Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#4534 closed bug (fixed)

UI Dialog causing iframe to reload content every time 'show' is called

Reported by: bgrinstead Owned by:
Priority: minor Milestone: 1.8.6
Component: ui.dialog Version: 1.7.1
Keywords: iframe show dialog Cc:
Blocked by: Blocking:

Description

I recently ran into this problem. Basically, if you have an iframe inside your dialog content, every time you reshow the dialog, it reloads the iframe. This is unexpected and incorrect behavior. I have tracked down the change in the open function:

(uiDialog.next().length && uiDialog.appendTo('body'));

This line works - the iframe does not reload because the DOM is not being manipulated, I guess?

(uiDialog.parent().not('body').length && uiDialog.appendTo('body'));

Note: I found the fix here: http://www.nabble.com/-jquery.ui.dialog--Dailog-with-a-lot-of-HTML-unbearably-slow-td21257084s27240.html

Please take a look at this!

Thanks, Brian Grinstead

Change History (6)

comment:1 Changed 8 years ago by Jörn Zaefferer

Milestone: TBD1.8

comment:2 Changed 8 years ago by burnrbr

I experienced the same issue in Firefox.

I changed this code:

(uiDialog.next().length && uiDialog.appendTo('body'));

To:

if (uiDialog.next().length && uiDialog.get(0).parentNode != document.body) {

uiDialog.appendTo('body');

}

comment:4 in reply to:  description Changed 7 years ago by watanabe

comment:5 Changed 7 years ago by Scott González

Milestone: 1.next1.9
Resolution: fixed
Status: newclosed

Fixed in e66cdfc.

comment:6 Changed 7 years ago by Scott González

Milestone: 1.91.8.6
Note: See TracTickets for help on using tickets.