Search and Top Navigation
#4898 closed bug (fixed)
Opened October 07, 2009 04:21AM UTC
Closed January 30, 2010 01:21AM UTC
$.ui.dialog.overlay: Too many events disabled
| Reported by: | mkemmerling | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.8 |
| Component: | ui.dialog | Version: | 1.7.2 |
| Keywords: | Cc: | markus.kemmerling@meduniwien.ac.at | |
| Blocked by: | Blocking: |
Description
The $.ui.dialog.overlay widget disables events on base of the event target's parent '.ui-dialog' element's z-index. This will also disable key events whose target happen to be the '.ui-dialog' itself, i.e copying with <CTL>-C. This can easily be verified with the modal dialog demo on the jQuery UI website.
Explicitely including the '.ui-dialog' element as possible event target fixes the problem. The attached ui.dialog.js patch adds .andSelf() at the approbiate position.
Attachments (1)
Change History (4)
Changed November 17, 2009 09:20AM UTC by comment:1
| milestone: | TBD → 1.8 |
|---|---|
| priority: | minor → major |
Changed December 08, 2009 10:00AM UTC by comment:2
We also had this problem with ui.dialog and copy-pasting under WinXP and FF.
As far as I known we have not seen these problems with WinXP and IE or Linux and FF, though.
I can confirm that the attached patch works as intended and solves copy-pasting from a ui.dialog with WinXP and FF.
Please fix this for release 1.8!
Changed January 29, 2010 10:19PM UTC by comment:3
This is also fixed by the fix to #4363, r3757. The zIndex() method checks the target as well as the parents to get the zIndex, so events directly on the ui.dialog will be allowed to proceed.
Changed January 30, 2010 01:21AM UTC by comment:4
| resolution: | → fixed |
|---|---|
| status: | new → closed |