Needs review
Project:
Thickbox
Version:
6.x-1.6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2011 at 15:33 UTC
Updated:
20 Dec 2011 at 16:06 UTC
Jump to comment: Most recent file
Reproduce: ESC key doesn't work with IE9.
Reason: The condition to check for ie on line 279 is no longer valid: e == null
Patch: change the following line in the code (appears twice)
- escapeKey = e.DOM_VK_ESCAPE
+ escapeKey = e.DOM_VK_ESCAPE || 27
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | thickbox-esckey-ie9-1377598-01.patch | 750 bytes | madmanmax |
Comments
Comment #1
madmanmax commentedAttached patch.
Comment #2
madmanmax commentedRemoved tag... sorry.