Opened 10 years ago
Closed 10 years ago
#8838 closed bug (fixed)
Dialog: Close icon does not work in dialog larger than the window in IE
Reported by: | clifforama | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.10.0 |
Component: | ui.dialog | Version: | 1.9.1 |
Keywords: | regression | Cc: | |
Blocked by: | Blocking: |
Description
After opening a dialog that doesn't fit within the window, clicking the close icon won't close the dialog. Instead, it moves the dialog to the top of the viewport. Only when the dialog is top-aligned to the viewport will a subsequent click close the dialog.
Problem occurs with jQuery UI 1.9.1, but does *NOT* occur with jQuery UI 1.9.0.
Reproducible in:
- Internet Explorer 9
- Internet Explorer 9 in IE8 Browser Mode
- Internet Explorer 9 in IE7 Browser Mode
Not reproducible in:
- Firefox 16.0.2
- Chrome 23.0.1271.64 m
- Safari 5.1.7 (7534.57.2)
Test case:
- http://jsfiddle.net/zcYGE/10/
- reduce height of results panel to as small as possible
- click "Open dialog" link to open the dialog
- scroll results panel until dialog's close icon is in view
- click close icon (this moves dialog to top of results panel instead of closing dialog)
- click close icon again to actually close dialog
Change History (3)
comment:1 Changed 10 years ago by
Keywords: | regression added |
---|---|
Status: | new → open |
Summary: | In dialog that's larger than window, close icon only works when dialog is top-aligned to viewport → Dialog: Close icon does not work in dialog larger than the window in IE |
comment:2 Changed 10 years ago by
Sounds very similar to #8789. The fix here will probably fix both: https://github.com/jquery/jquery-ui/pull/828 Still not sure what exactly is happening.
comment:3 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Dialog: Don't focus dialog when mousedown is on close button. Fixes #8838 - Dialog: Close icon does not work in dialog larger than the window in IE.
Changeset: 60486ac632a0a1bbbb0c7449fe17bccfae11af80
Confirmed. The issue also occurs in IE10 and is not present in 1.9.0.