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

Alert notification don't close properly when multiple are opened

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.6, 2.6.1, 2.6.2
    • 2.5.6, 2.6.3
    • JavaScript
    • MOODLE_26_STABLE
    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • MDL-44855-master
    • 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

    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

              dobedobedoh Andrew Lyons
              sillyxone Huy Hoang
              Petr Skoda Petr Skoda
              Marina Glancy Marina Glancy
              Frédéric Massart Frédéric Massart
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              4 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                12/May/14