Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-39508

Some standard CORE themes do not display SCORM navigation drop down properly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4.3
    • Fix Version/s: 2.5
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      This should fix all CORE themes that use Canvas as parent theme, so test only those themes which are: arialist, binarius, boxxie, brick, formfactor, formal_white, fusion, leatherbound, magazine, nimble, nonzero, overlay, sky_high, serenity, splash:

      1. Login as a teacher.
      2. Update a SCORM activity and set 'Display course structure in player' to 'In a dropdown menu'.
      3. Check that the SCORM table of contents is displayed in a dropdown menu properly.
        MDLQA-5332 has screenshots of the issue.
      Show
      This should fix all CORE themes that use Canvas as parent theme, so test only those themes which are: arialist, binarius, boxxie, brick, formfactor, formal_white, fusion, leatherbound, magazine, nimble, nonzero, overlay, sky_high, serenity, splash: Login as a teacher. Update a SCORM activity and set 'Display course structure in player' to 'In a dropdown menu'. Check that the SCORM table of contents is displayed in a dropdown menu properly. MDLQA-5332 has screenshots of the issue.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-39508_master

      Description

      1. Login as a teacher, update a SCORM activity and set 'Display course structure in player' under 'Display settings (Show more...)' to Hidden.
      2. Check that the SCORM table of contents is no longer displayed on the SCORM outline page.
      3. Update the SCORM activity and set 'Display course structure in player' to 'In a dropdown menu'.
      4. Check that the SCORM table of contents is displayed in a dropdown menu properly.
      MDLQA-5332 has screenshots of the issue.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              This issue was raised during a QA test, but through later discussion it was decided that this was not a significant enough issue to block a release.

              I've adjusted the priority accordingly and hopefully we will be able to get to this issue soon.

              Show
              salvetore Michael de Raadt added a comment - This issue was raised during a QA test, but through later discussion it was decided that this was not a significant enough issue to block a release. I've adjusted the priority accordingly and hopefully we will be able to get to this issue soon.
              Hide
              lazydaisy Mary Evans added a comment -

              Can anyone tell me if the same SCORM package works in ALL other Canvas based theme or is this JUST a leatherbound BUG?

              Also, so that I can jet a hold of the problem can I test it in the QA site? It would be easier fo me to find a fix if it was. Thanks

              Show
              lazydaisy Mary Evans added a comment - Can anyone tell me if the same SCORM package works in ALL other Canvas based theme or is this JUST a leatherbound BUG? Also, so that I can jet a hold of the problem can I test it in the QA site? It would be easier fo me to find a fix if it was. Thanks
              Hide
              jasmink Jasmin Klindzic added a comment -

              Mary, AL Rachels tested some other themes (and it works with them) while testing MDLQA-5332, you can check it out here (screenshots available): https://tracker.moodle.org/browse/MDLQA-5332

              Show
              jasmink Jasmin Klindzic added a comment - Mary, AL Rachels tested some other themes (and it works with them) while testing MDLQA-5332 , you can check it out here (screenshots available): https://tracker.moodle.org/browse/MDLQA-5332
              Hide
              lazydaisy Mary Evans added a comment -

              Test instructions are aimed as a SCORM Admin! not a Dummy like me! LOL

              Show
              lazydaisy Mary Evans added a comment - Test instructions are aimed as a SCORM Admin! not a Dummy like me! LOL
              Hide
              lazydaisy Mary Evans added a comment -

              OK...I think I have it fixed now...in my head at least!

              Thanks

              Show
              lazydaisy Mary Evans added a comment - OK...I think I have it fixed now...in my head at least! Thanks
              Hide
              lazydaisy Mary Evans added a comment -

              So I don't forget:
              The CSS path to this is as follows:

              html.yui3-js-enabled body#page-mod-scorm-player.format-weeks div#page div#page-content-wrapper.wrapper div#page-content div#region-main-box div#region-post-box div#region-main-wrap div#region-main div.region-content div div#scormpage div#tocbox div#scormtop div#scormnav.scorm-right div.singleselect form#tocmenu div select#single_select5187eddbe3ba31.select

              Leatherbound adds a 1em margin top and bottom to the single_select form element and so pushes it behind the SCORM player.

              Show
              lazydaisy Mary Evans added a comment - So I don't forget: The CSS path to this is as follows: html.yui3-js-enabled body#page-mod-scorm-player.format-weeks div#page div#page-content-wrapper.wrapper div#page-content div#region-main-box div#region-post-box div#region-main-wrap div#region-main div.region-content div div#scormpage div#tocbox div#scormtop div#scormnav.scorm-right div.singleselect form#tocmenu div select#single_select5187eddbe3ba31.select Leatherbound adds a 1em margin top and bottom to the single_select form element and so pushes it behind the SCORM player.
              Hide
              lazydaisy Mary Evans added a comment - - edited

              I'm finding that this is OK in Base theme, Afterburner and Anomaly, but has a problem with all the other standard themes that inherit a setting from Canvas theme for .singleselect that adds a 1em margin to that dropdown menu. So really the original test should have failed. I think that is what the thought was at the time. The fix needs to go into Canvas theme, which in turn will fix all the other themes.

              Show
              lazydaisy Mary Evans added a comment - - edited I'm finding that this is OK in Base theme, Afterburner and Anomaly, but has a problem with all the other standard themes that inherit a setting from Canvas theme for .singleselect that adds a 1em margin to that dropdown menu. So really the original test should have failed. I think that is what the thought was at the time. The fix needs to go into Canvas theme, which in turn will fix all the other themes.
              Hide
              poltawski Dan Poltawski added a comment -

              Integrated to master, thanks Mary

              Show
              poltawski Dan Poltawski added a comment - Integrated to master, thanks Mary
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              tested all themes mentioned in the instructions, they all show the drop down correctly now.
              Passing
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - tested all themes mentioned in the instructions, they all show the drop down correctly now. Passing Thanks
              Hide
              dmonllao David Monllaó added a comment -

              Passing according to Ankit's comments, probably he forgot pressing the button

              Show
              dmonllao David Monllaó added a comment - Passing according to Ankit's comments, probably he forgot pressing the button
              Hide
              lazydaisy Mary Evans added a comment -

              Show
              lazydaisy Mary Evans added a comment -
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Did you think this day was not going to arrive ever?

              Your patience has been rewarded, yay, sent upstream, thanks!

              Closing...ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/May/13