Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.6
-
Fix Version/s: 2.6.2
-
Component/s: JavaScript
-
Testing Instructions:
-
Affected Branches:MOODLE_26_STABLE
-
Fixed Branches:MOODLE_26_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-43979-master -
Pull Master Diff URL:
Description
If you create a notification that is not initially visible, then use the 'show' method to show it, the z-index for the dialog is not set.
The easiest way to see this is with the glossary dialog. First set up a glossary to autolink and make a page that contains something linked. Then, two ways to reproduce:
1. Put something on the screen including the bit where the glossary dialog will appear, which has a z-index e.g. 10 and moodle-has-zindex class (meaning that the glossary should automatically appear on top of it). Click the glossary link. Observe that the glossary popup shows UNDER the z-indexed item.
or
2. Edit lib/yui/src/notification/js/dialogue.js to add a console.log line to the start of applyZIndex function (or add a breakpoint in your browser). Click the glossary link. Observe that applyZIndex is not called.
This problem occurs because applyZIndex runs in two situations, either when you create something with
{visible:true}, or when the visibilityChanged function is called. This does not happen if you create something initially not visible, then call the show() method, which is what glossary does.
Attachments
Issue Links
- blocks
-
MDL-44161 Modal Dialogues cause a jump in page position
-
- Closed
-