This is a follow-up to #517688: Initial D7UX admin overlay: we need to fix how the size of the iframe is recomputed when navigating to a new page inside the overlay. Currently, the iframe container collapse itself for a brief moment before the content of the target page is loaded and processed. This is very unpleasant and looks amateur.

Comments

cwgordon7’s picture

cwgordon7 CreditAttribution: cwgordon7 commented
Assigned: Unassigned » cwgordon7

I'll agree to do this once the main overlay patch gets in. It will be much pleasanter to work with smaller patches. :)

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

subscribing

seutje’s picture

seutje CreditAttribution: seutje commented

subscribe

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Title: Overlay: fix transitions between pages inside the overlay » Fix transitions between pages inside the overlay
Component: base system » overlay.module

Still seems to be an issue, so I'm moving it to the correct queue.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented

#664450: Heavy flicker effect due to white background rebuilding every time when going to new page within overlay is probably a duplicate but has a nice series of screenshots illustrating the problem.

It was also pointed out, though, that this behavior may wind up being fixed by the latest patches at #615130: Overlay locks up the browser and consumes 100% of CPU for certain browsers/graphics cards/operating systems.

casey’s picture

casey CreditAttribution: casey commented
Status: Active » Fixed

This is fixed per #174 in #615130: Overlay locks up the browser and consumes 100% of CPU for certain browsers/graphics cards/operating systems.

Anyone who can still reproduce it is free to reopen.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Closed (fixed) » Active

This was fixed for a while, but I'm seeing it again on the latest HEAD. Anyone else?

(If it's already being worked on somewhere else, feel free to close this again and post a link to that issue.)

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

Do you mean this issue David?

#679830: IE8 doesn't show content on the "manage field" page

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Active » Closed (fixed)

Ah, yes, I think that's it. Sorry for missing that one.