Dialog: Modal dialogs do not disable resizables on the page

I can confirm the bug and that the z-index difference is causing this issue.  Simply changing the z-index declaration on line 11 of jquery.ui.resizable.css to "999" instead of "99999" fixes this particular situation.

However, since z-index is a configurable parameter of both dialog and resizable, this type of solution probably isn't going to work.

The only other thing I could think of was to add a marker class to resizable elements when a modal is opened and remove the class when the dialog is closed.  But when you start to think about the fact that the dialog itself can be resizable, and a dialog can technically have resizable elements within it, and that you can have multiple dialogs open at once... this type of solution gets really messy.

Since the zIndex is a configurable option for both of these, personally I think the best way around this is to explicitly pass in the zIndex values that you'd like to use for each individual call. Otherwise this will quickly turn into a zIndex arms race.

The one issue with this is that currently, the value passed into the resizable plugin will only be applied to the corners. The non-corner handles will take the z-index of 99999 from CSS. Lines 106-108 of jquery.ui.resizable.js:

// increase zIndex of sw, se, ne, nw axis
//TODO : this modifies original option
if(/sw|se|ne|nw/.test(handle)) axis.css({ zIndex: ++o.zIndex });

I'm not sure why this is, it seems like you'd want the same zIndex values on all of the sides for consistency and to avoid issues like this one, maybe I'm wrong. I submitted a pull request to change apply the zIndex passed in to all handles - https://github.com/jquery/jquery-ui/pull/561

Landed the resizable change in 6150abae49f77a3b2f67c39a3c60a06792b4de8f.

tj.vantoll, could you retest this against master? The overlay changes should have fixed this.

joern.zaefferer Looks like this is still an issue in master - http://jsfiddle.net/tj_vantoll/TmmQ7/.

.ui-front

has a

z-index

of 100 and the

resizable

handles have a

z-index

of 1000. If I pass in

zIndex: 99

to

resizable

on the above fiddle the

resizable

can no longer be resized.

Yeah, this is purely a z-index problem. We should fix it by changing resizable handles to something smaller, like 90.

Do we need a separate resizable ticket for a change like that? Or would we commit that resizable change pointing at this dialog ticket?

We can just reference this ticket. It'd be one of the few commits that has a different component listed at the front of the commit and in the ticket reference.

https://github.com/jquery/jquery-ui/blob/master/ui/jquery.ui.resizable.js#L45.

1000 --> 90?

Can anybody think of any adverse effects of that change? Otherwise I can send a pull request.

That should be all that's required. The only adverse effect would be users who have other elements with high z-indexes that overlap resize handles, which seems like it would probably result in bad UX anyway (and it's super easy for the user to fix).

Pull request - https://github.com/jquery/jquery-ui/pull/840.

Resizable: Modified the default z-index value of resizable handles. Fixed #7960 - Dialog: Modal dialogs do not disable resizables on the page.

Changeset: 0cd470b0d5f8c617e761c4a6c90aeea4e3f54128