New stacking feature causing iframe problems when changing active dialog

[aNt1X]

Hi,

Talking about the following commit:

​https://github.com/jquery/jquery-ui/commit/3829a37ca122e923c3a08b964c4b1a946a2a1456

The new moveToTop() function breaks iframe usage because of this line of code:

var moved = this.uiDialog.nextAll( ":visible" ).insertBefore( this.uiDialog );

The insertBefore() usage makes the iframe to be detached and reattached from the DOM; you can have occasional javascript errors or entire iframe refresh, depending on which browser you use; i think it is because of this:

​http://bugs.jquery.com/ticket/8533

What do u think?

I know it's not a jQuery ui problem, but how can we avoid this ?

Thank you for your work.

[aNt1X]

Sorry guys, i left the component setting of this ticket to "ui.core" but it is "ui.dialog".

Sorry again.

[Scott González]

I think browser vendors should fix this. Do you have any suggestions?

[aNt1X]

Not yet. I will further investigate it.

Vendors implemented this protection to avoid memory leaks (​http://msdn.microsoft.com/en-us/library/gg622929(v=VS.85).aspx?ppud=4).

We could modify the z-index only for those dialogs that contains an iframe, leaving the new mechanism for dialogs without an iframe, but it would be an ugly workaround, and the old z-indexes method caused many problems ...

[mandana]

I have the same problem after upgrading JqueryUI to 1.10 As there is no other way to move an ifram in DOM without loosing it's content, using z-index seems to be the only possible solution to this issue. BTW I don't think of it as a minor bug!!!

[mikesherov]

Nothing we can do here... Although, fortunately, now jQuery has a meta ticket in webkit for issues we need fixed. This has been added.

Commented here: ​https://bugs.webkit.org/show_bug.cgi?id=13574 and noted as a blocker here: ​https://bugs.webkit.org/show_bug.cgi?id=110007

[Jörn Zaefferer]

This is a duplicate of #9166 (or the other way round), which we're addressing on our end, since browsers declared WONTFIX. So the cantfix solution isn't correct anymore.

[Jörn Zaefferer]

Duplicate of #9166.