Ticket #4898 (closed bug: fixed)

Opened 4 years ago

Last modified 3 years ago

$.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@…
Blocking: Blocked by:

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

ui.dialog.js.patch Download (544 bytes) - added by mkemmerling 4 years ago.

Change History

Changed 4 years ago by mkemmerling

comment:1 Changed 4 years ago by joern.zaefferer

  • Priority changed from minor to major
  • Milestone changed from TBD to 1.8

comment:2 Changed 3 years ago by duelli

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!

comment:3 Changed 3 years ago by btburnett3

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.

comment:4 Changed 3 years ago by scott.gonzalez

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.