Closed Bug 198479 Opened 22 years ago Closed 22 years ago

IFrame doesn't hide when setting visibility:hidden (breaks Midas demo)

Categories

(Core :: Web Painting, defect)

x86
Windows XP
defect
Not set
major

Tracking

()

RESOLVED FIXED

People

(Reporter: emaijala+moz, Assigned: roc)

References

Details

(Keywords: regression, Whiteboard: [fix])

Attachments

(2 files)

IFrame, once shown, will not hide. Setting visibility:hidden only hides its frame, but the content remains. The problem is apparent in the color pickers of Midas demo, but I'll attach a pretty minimal test case too.
Attached file test case
A simple test case that reportedly works in 1.3 (I haven't tested). I see the problem at least in 2003031808, but it might be older.
Summary: IFrame doesn't hide when setting visibility:hidden → IFrame doesn't hide when setting visibility:hidden (breaks Midas demo)
Blocks: 198472
This is a major problem when implementing Midas.
Severity: normal → major
Flags: blocking1.4a?
Keywords: regression
Flags: blocking1.4a? → blocking1.4a-
Status: NEW → ASSIGNED
Whiteboard: [fix]
Attached patch fixSplinter Review
This broke when the logic for hiding views in response to CSS "visibility:hidden" was loosened up. For IFRAMEs we want to hide the view in that case, because its content will not obey our CSS "visibility:hidden".
Attachment #119595 - Flags: superreview?(bzbarsky)
Attachment #119595 - Flags: review?(bzbarsky)
Attachment #119595 - Flags: superreview?(bzbarsky)
Attachment #119595 - Flags: superreview+
Attachment #119595 - Flags: review?(bzbarsky)
Attachment #119595 - Flags: review+
checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
*** Bug 203043 has been marked as a duplicate of this bug. ***
*** Bug 198472 has been marked as a duplicate of this bug. ***
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: