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:
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-44855-master
    • 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

      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 Lyons
              Reporter:
              sillyxone Huy Hoang
              Peer reviewer:
              Petr Skoda Petr Skoda
              Integrator:
              Marina Glancy Marina Glancy
              Tester:
              Frédéric Massart Frédéric Massart
              Participants:
              Component watchers:
              Andrew Lyons, Dongsheng Cai, 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:
                Fix Release Date:
                12/May/14