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

Recast SCORM style.css in block style

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      This is a change to whitespace in the code only - no actual UI changes should occur.

      it should be sufficient to open a scorm and make sure it displays the same way it did before the patch.

      Show
      This is a change to whitespace in the code only - no actual UI changes should occur. it should be sufficient to open a scorm and make sure it displays the same way it did before the patch.
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:

      Description

      The mod/scorm/style.css is to be rewritten as per block style CSS coding style mentioned here
      http://docs.moodle.org/dev/CSS_coding_style#Block_Style

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              danmarsden Dan Marsden added a comment -

              just did this as part of the work I'm doing in MDL-41290

              Show
              danmarsden Dan Marsden added a comment - just did this as part of the work I'm doing in MDL-41290
              Hide
              danmarsden Dan Marsden added a comment -

              MDL-41290 must be integrated before this patch.

              Show
              danmarsden Dan Marsden added a comment - MDL-41290 must be integrated before this patch.
              Hide
              danmarsden Dan Marsden added a comment -

              this will conflict - re-opening until next week after MDL-39910 is sorted.

              Show
              danmarsden Dan Marsden added a comment - this will conflict - re-opening until next week after MDL-39910 is sorted.
              Hide
              cibot CiBoT added a comment -

              Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

              Show
              cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
              Hide
              nobelium vignesh p added a comment -

              Hi Dan, I saw your branch n its kinda obsolete. I have edited style.css on my repository. https://github.com/nobelium/moodle/compare/MDL-41216. If everything is ok please merge it.

              Show
              nobelium vignesh p added a comment - Hi Dan, I saw your branch n its kinda obsolete. I have edited style.css on my repository. https://github.com/nobelium/moodle/compare/MDL-41216 . If everything is ok please merge it.
              Show
              cibot CiBoT added a comment - Results for MDL-41216 Remote repository: https://github.com/nobelium/moodle.git Remote branch MDL-41216 to be integrated into upstream master Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/2300 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/2300/artifact/work/smurf.html
              Hide
              danmarsden Dan Marsden added a comment -

              thanks Vignesh - looks good to me.

              Show
              danmarsden Dan Marsden added a comment - thanks Vignesh - looks good to me.
              Hide
              cibot CiBoT added a comment -

              Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

              Show
              cibot CiBoT added a comment - Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!
              Hide
              cibot CiBoT added a comment -

              Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

              Show
              cibot CiBoT added a comment - Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated (master), thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (master), thanks!
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              I've tried a simple SCORM and also have compared the before/after css with recess and both are 100% the same.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - I've tried a simple SCORM and also have compared the before/after css with recess and both are 100% the same.
              Hide
              marina Marina Glancy added a comment -

              Thanks for your awesome code, it is now part of Moodle. Don't forget to submit another issue next week (and peer review two).

              Show
              marina Marina Glancy added a comment - Thanks for your awesome code, it is now part of Moodle. Don't forget to submit another issue next week (and peer review two).

                People

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

                  Dates

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