Opened 12 years ago
Closed 11 years ago
#23370 closed enhancement (fixed)
Update to jQuery UI 1.10.*
Reported by: | ocean90 | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | External Libraries | Keywords: | has-patch |
Focuses: | Cc: |
Description
Release post: http://blog.jqueryui.com/2013/01/jquery-ui-1-10-0/
Attachments (7)
Change History (28)
#3
in reply to:
↑ 1
@
12 years ago
- Keywords needs-patch added; has-patch needs-testing removed
Replying to ocean90:
Because of the Dialog API Redesign (Upgrade Guide) there is an issue with the edit/insert link dialog, see http://cl.ly/Md0u
#4
@
12 years ago
- Keywords has-patch added; needs-patch removed
- Summary changed from Update to jQuery UI 1.10.0 to Update to jQuery UI 1.10.*
23370.2.patch: jQuery UI 1.10.1
Dialog unchanged.
#6
@
12 years ago
- Syncs the jQuery UI CSS styles with the current one
- Hardcodes the old z-index values, since the zIndex attribute is gone
- Fixes the close button
Patch tested in Chrome, Opera, Safari Firefox, IE 7-10. RTL tested too.
#7
@
12 years ago
Another issue:
Open DFW and switch to text mode. Add a link. Link will be added but you get an error too:
Uncaught TypeError: Cannot call method '_focusTabbable' of undefined j query.ui.dialog.min.js:5 (anonymous function) jquery.ui.dialog.min.js:5 jQuery.event.dispatch jquery.js:3074 elemData.handle jquery.js:2750 jQuery.event.trigger jquery.js:2986 jQuery.event.trigger jquery-migrate.js:468 jQuery.event.simulate jquery.js:3301 handler jquery.js:3552 wpLink.htmlUpdate wplink.js:212 wpLink.update wplink.js:158 (anonymous function) wplink.js:35 jQuery.event.dispatch jquery.js:3074 elemData.handle
#9
@
12 years ago
Is there an upstream regression fixed by 23370.4.patch?
Is the issue in comment 7 still an issue in 1.10.1? If so, do we need to report something upstream?
#10
@
12 years ago
Is there an upstream regression fixed by 23370.4.patch?
Yesno. They removed support for a zindex property.
Is the issue in comment 7 still an issue in 1.10.1?
Yes.
If so, do we need to report something upstream?
#11
@
12 years ago
Also, here is where zindex was removed. Not a major issue, they just no longer wanted to juggle z-index and would prefer that it is left to CSS. http://bugs.jqueryui.com/ticket/8729
#16
@
11 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 23892:
#17
follow-up:
↓ 18
@
11 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
This is causing a z-index conflict in /wp-includes/css/jquery-ui-dialog.css:
.ui-widget-overlay { z-index: 300001;
is over
.ui-front { z-index: 100;
A new dialog has its shadow in front of it, you can't click 'close'. For example, try this on a wp-admin page:
var msg = '<div>hello</div>'; jQuery(msg).dialog({ draggable: false, resizable: false, modal: true });
The close button and the form are behind the overlay.
#18
in reply to:
↑ 17
@
11 years ago
i checked into the issue you reported on the dialog not loading correctly. your code indeed shows the dialog behind the overlay, oops! i figured out you need do do something slightly different, nothing is wrong with the CSS!
not sure if this is documented anywhere, but it looks like you need to add the wp-dialog class to your dialogs to get them to show up properly in the admin interface, try this update of your example to see what happens, you should see the modal:
var msg = '<div>hello this is a test</div>';
jQuery(msg).dialog({ draggable: false, resizable: false, modal: true, dialogClass: 'wp-dialog' });
Replying to programmin:
This is causing a z-index conflict in /wp-includes/css/jquery-ui-dialog.css:
.ui-widget-overlay { z-index: 300001;is over
.ui-front { z-index: 100;A new dialog has its shadow in front of it, you can't click 'close'. For example, try this on a wp-admin page:
var msg = '<div>hello</div>'; jQuery(msg).dialog({ draggable: false, resizable: false, modal: true });The close button and the form are behind the overlay.
Because of the Dialog API Redesign (Upgrade Guide) there is an issue with the edit/insert link dialog, see http://cl.ly/Md0u