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

Chooser Dialogue does not cancel listen events when closed

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: JavaScript
    • Labels:
    • Testing Instructions:
      Hide
      • Open a course
      • Turn editing on
      • Open the activity chooser
        • Ensure that the scrollbars are locked
      • Close the activity chooser
        • Ensure that the scrollbars are unlocked
      • Resize your browser window (or rotate your device)
        • Ensure that the scrollbars are unlocked
      • Open the chooser
        • Ensure that it still opens
        • Ensure that the scrollbars are locked
      • Close the chooser
      Show
      Open a course Turn editing on Open the activity chooser Ensure that the scrollbars are locked Close the activity chooser Ensure that the scrollbars are unlocked Resize your browser window (or rotate your device) Ensure that the scrollbars are unlocked Open the chooser Ensure that it still opens Ensure that the scrollbars are locked Close the chooser
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-33891-master-1

      Description

      This is an oversight on my part from when I added the event canceller to the chooser dialogue.
      This idea of the cancel_listeners function is to cancel all events relating to an open dialogue. This includes things such as window resizing.
      Now that MDL-33649 has been integrated, it means that if you resize the window after you close the dialogue (or if you're IE8 when you close the dialogue) the scrollbars are locked.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Integrated, thanks

            Show
            poltawski Dan Poltawski added a comment - Integrated, thanks
            Hide
            nebgor Aparup Banerjee added a comment -

            for Fred

            Show
            nebgor Aparup Banerjee added a comment - for Fred
            Hide
            fred Frédéric Massart added a comment -

            Tested passed on IE8 and Chrome. Now MDL-33649 can be passed too!

            Show
            fred Frédéric Massart added a comment - Tested passed on IE8 and Chrome. Now MDL-33649 can be passed too!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay!

            Many, many thanks for your hard work!

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay! Many, many thanks for your hard work! Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Jun/12