Details

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

      Can only really be tested/verified by a dev - you could just open a SCORM package and make sure it resizes correctly when you change the size of the window...

      Show
      Can only really be tested/verified by a dev - you could just open a SCORM package and make sure it resizes correctly when you change the size of the window...
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-28601
    • Rank:
      18278

      Description

      since MDL-22951, rd.js file in mod/scorm doesn't get used anymore - it should be removed.

        Activity

        Hide
        Dan Marsden added a comment -

        Note to integrator - feel free to cherry pick onto 2.1 and 2.0stable branches if you feel like it... doesn't worry me either way. This file should have been removed before 2.0 was released.

        Show
        Dan Marsden added a comment - Note to integrator - feel free to cherry pick onto 2.1 and 2.0stable branches if you feel like it... doesn't worry me either way. This file should have been removed before 2.0 was released.
        Hide
        Petr Škoda added a comment -

        Integrated, I did not backport it because many admins will only override the files during upgrades, so there is no way to force it out everywhere.

        Thanks.

        Show
        Petr Škoda added a comment - Integrated, I did not backport it because many admins will only override the files during upgrades, so there is no way to force it out everywhere. Thanks.
        Hide
        Aparup Banerjee added a comment -

        yup resizing works (not till when its a tiny window but who does that..) and the file is gone!

        Show
        Aparup Banerjee added a comment - yup resizing works (not till when its a tiny window but who does that..) and the file is gone!
        Hide
        Dan Marsden added a comment -

        Thanks Apu - there's a min size in the code somewhere to help prevent some of the mess that could cause!

        Show
        Dan Marsden added a comment - Thanks Apu - there's a min size in the code somewhere to help prevent some of the mess that could cause!
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Sent upstream and closing, many thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: