Search and Top Navigation
#9067 closed bug (duplicate)
Opened February 07, 2013 02:48PM UTC
Closed September 30, 2013 09:29AM UTC
New stacking feature causing iframe problems when changing active dialog
| Reported by: | aNt1X | Owned by: | aNt1X |
|---|---|---|---|
| Priority: | minor | Milestone: | none |
| Component: | ui.dialog | Version: | 1.10.0 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
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.
Attachments (0)
Change History (7)
Changed February 07, 2013 02:49PM UTC by comment:1
Changed February 07, 2013 02:53PM UTC by comment:2
| component: | ui.core → ui.dialog |
|---|---|
| owner: | → aNt1X |
| status: | new → pending |
I think browser vendors should fix this. Do you have any suggestions?
Changed February 07, 2013 03:36PM UTC by comment:3
| status: | pending → new |
|---|
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 ...
Changed February 11, 2013 06:43AM UTC by comment:4
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!!!
Changed February 22, 2013 08:20PM UTC by comment:5
| resolution: | → cantfix |
|---|---|
| status: | new → closed |
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
Sorry guys, i left the component setting of this ticket to "ui.core" but it is "ui.dialog".
Sorry again.