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

Alert notification don't close properly when multiple are opened

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6, 2.6.1, 2.6.2
    • Fix Version/s: 2.5.6, 2.6.3
    • Component/s: JavaScript
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a glossary with autolink enabled
      2. Add a number of glossary links on a theme. Have each one reference some of the others. I use the theme of fruit (Fruit -> Apple -> Orange -> etc)
      3. Enable the glossary autolink filter
      4. Add some text in the course which contains one of the glossary words
      5. Click on the glossary link, then open a few more of the dialogues
      6. Close them one by one
        • Confirm that they all close correctly and in order
      Show
      Create a glossary with autolink enabled Add a number of glossary links on a theme. Have each one reference some of the others. I use the theme of fruit (Fruit -> Apple -> Orange -> etc) Enable the glossary autolink filter Add some text in the course which contains one of the glossary words Click on the glossary link, then open a few more of the dialogues Close them one by one Confirm that they all close correctly and in order
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-44855-master

      Description

      In the Glossary, when clicking on the terms to open pop-ups without closing them, only the last pop-up can then be closed. This is caused by the YUI Alert module (lib/yui/src/notification/js/alert.js) storing all the event handlers into one array and detach them all when a pop-up is closed.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dobedobedoh Andrew Nicols
              Reporter:
              sillyxone Huy Hoang
              Peer reviewer:
              Petr Skoda
              Integrator:
              Marina Glancy
              Tester:
              Frédéric Massart
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              4 Vote for this issue
              Watchers:
              10 Start watching this issue

                Dates

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