Closed Bug 20185 Opened 25 years ago Closed 25 years ago

absolute position elements repaint wrong w/GFX scrollbars on

Categories

(Core :: Web Painting, defect, P2)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: beard)

References

()

Details

(Keywords: platform-parity, Whiteboard: [PDT+] was fixed with viewmanager2)

Attachments

(3 files)

DESCRIPTION: Pages with absolutely positioned elements (these have their own views, I believe) are not repainting correctly. This shows up on my homepage, http://www.fas.harvard.edu/~dbaron/ , where: * scrolling down and back up causes the header at the top to disappear (and be replaced by light gray) * switching to another virtual screen and back causes the entire viewport to be light gray A simplified testcase will also be attached, where the page in question displays nothing. (The correct display flashes quickly by before the screen goes blank.) STEPS TO REPRODUCE: * load testcase ACTUAL RESULTS: * blank EXPECTED RESULTS: * the letters A, B, and C, not necessarily in that order DOES NOT WORK CORRECTLY ON: * Linux, mozilla, 1999-11-28-08-M12 WORKS CORRECTLY ON: * builds more than a week before (I haven't tested between) ADDITIONAL INFORMATION: I'm marking this DOGFOOD because it prevents me from using my homepage. It likely affects other pages with absolute positioning (e.g., hotbot), although I haven't checked. (It hit 2 of 2 pages with absolute positioning that I tried.) It's also a major regression.
Attached file testcase described in bug (deleted) —
ALTERNATE STEPS TO REPRODUCE: * load http://www.hotbot.com/ * resize the window ACTUAL RESULTS: * all of the page except for the animated GIF advertisement disappears This might be Linux-only. I can't tell...
Whiteboard: [PDT+]
Putting on PDT+ radar.
Status: NEW → ASSIGNED
Target Milestone: M12
Definitely linux-only regression. I don't see this on the Mac.
No ETA on this yet. I need a linux helper to diagnose this.
Summary: [DOGFOOD]page with absolutely positioned elements not repainting correctly → [DOGFOOD][PP]page with absolutely positioned elements not repainting correctly
Marking [PP] since this is Linux-only. This gets fixed when I turn GFX scrollbars off. cc:ing evaughan. However, it could be another bug that only showed up once GFX scrollbars were turned on... Could it be related to bug 19142?
Assignee: beard → pavlov
Status: ASSIGNED → NEW
The page and test case work on Mac. Handing to pavlov to look into problems on linux.
Whiteboard: [PDT+] → [PDT+] 12/10/99
Assignee: pavlov → evaughan
Summary: [DOGFOOD][PP]page with absolutely positioned elements not repainting correctly → [DOGFOOD][PP]page with absolutely positioned elements not repainting correctly with GFX scrollbars on
i see it with gfx scrollbars on but it is fine without them. reassigning to eric
Hmmm. I wonder if Pavlov's fix to bug 20496 could've helped this? I guess I can check for myself tomorrow evening...
I think this might be the same as the bug that I've been chasing for the last couple of days: http://bugzilla.mozilla.org/show_bug.cgi?id=18802
*** Bug 18802 has been marked as a duplicate of this bug. ***
If I turn the #ifdef NEW_COMPOSITOR to an #if 0 in nsViewManager::RenderViews this problem goes away. Looks like a problem in that compositor code. Also, this test case works _so_ much better than the test cases that I had.
Assignee: evaughan → blizzard
Woah. Why is this assigned to me? I'm clueless about the compositor but I'll try to track it down. Someone in layout should look at it, really.
Readding evaughan to cc: list. Note Chris Blizzard's last comment. If this isn't fixed for M12, then, IMO, you should consider turning GFX scrollbars off by default for the Linux M12 release.
...or use the workaround that I've mentioned above unless that causes more problems than it solves.
Assignee: blizzard → evaughan
Whiteboard: [PDT+] 12/10/99 → [PDT+]
This should go to eric for examination. Reassigning, and clearing fix estimate date.
Assignee: evaughan → pavlov
Priority: P3 → P2
Summary: [DOGFOOD][PP]page with absolutely positioned elements not repainting correctly with GFX scrollbars on → [DOGFOOD][PP]absolute position elements repaint wrong w/GFX scrollbars on
Whiteboard: [PDT+] → [PDT+] needs estimated fix date (a Wed.or a Fri.)
Rick, could you comment on Blizzard's assertion that NEW_COMPOSITOR is also a necessary precondition, and his proposed workaround? In the meantime, assigning to pavlov, who will work with evaughan on this. p2 for m12 If this only affects a small number of pages, why is it PDT+ dogfood?
Blocks: 21564
Whiteboard: [PDT+] needs estimated fix date (a Wed.or a Fri.) → [PDT+] 12/17/99
I see this on http://www.nokia.com/phones/8210/main.html, if I comment out all <INPUT type="image" ...> on a local copy of that page the problem disappears (no absolute position involved AFAIK)... I see this problem only on linux with GFX scrollbars turned on. With blizzards workarroud the page looks OK, with or without GFX scrollbars.
Target Milestone: M12 → M13
pav sez, down deep in ugly code. hoping for early m13.
I've seen this with my workaround so I think it's a fluke that it works at all. Also, I've verified that it's an img submit button problem. It makes many pages completely unusable. Here are some: http://cnn.com/ http://www.abcnews.com/ http://www.etrade.com/ http://www.redhat.com/ http://www.linux.com/ This was just in 10 minutes of browsing. I beg. I plead. Please fix this for M12.
Blizzard - what do you mean by "I've verified that it's an img submit button problem." Do you mean that image submit buttons are a sufficient condition to cause the problem? They certainly aren't a necessary condition, because absolutely positioned elements are also a sufficient condition (on their own).
I meant that I've been able to verify that it happens on pages that have img buttons on them and the problem happens when those images come into view. It may or may not be related to the problem with elements. It might be a seperate bug.
*** Bug 20377 has been marked as a duplicate of this bug. ***
Blocks: 22176
Status: NEW → ASSIGNED
Whiteboard: [PDT+] 12/17/99 → [PDT+] 12/23/99
*sigh* this bug sucks. i'm working on it.
*** Bug 22520 has been marked as a duplicate of this bug. ***
Whiteboard: [PDT+] 12/23/99 → [PDT+] 01/14/00
blizzard? got any new ideas on this bug?
Nope. I tracked it down deep into the view manager.
i simply took a screenshot of http://www.nokia.com/phones/8210/main.html is it really what you are discussing here ? the same kind of grey areas appear on David Baron's homepage for me too.
Well, you don't need absolute positioning *or* an image submit button to trigger this bug. Observe <URL:http://www.openvrml.org>. The left frame typically has this drawing problem on the first load. The top frame grays out when I increase the browser window width beyond a certain threshold. I'll attach a screen shot.
That has absolutely position elements. The left frame has html in it like this: <DIV STYLE="position:absolute; width:130px; height:53; z-index:1; left: 0; top: 0; visibility: visible; overflow: visible"><A HREF="news.html" TARGET="main"><IMG SRC="pics/menunews.gif" WIDTH="130" HEIGHT="53" BORDER="0" ALT="NEWS"></A></DIV>
*** Bug 18750 has been marked as a duplicate of this bug. ***
*** Bug 22105 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [PDT+] 01/14/00 → [PDT+] 01/16/00
beard just checked in a fix. he rocks.
Blocks: 24206
Status: RESOLVED → VERIFIED
Great!!! It's fixed now!
Blocks: 24854
This one's back, reopening. I see the exact same problems in todays build on linux :(
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
*** Bug 23115 has been marked as a duplicate of this bug. ***
Adding dogfood and pp to keywords.
Keywords: dogfood, pp
Why does this bug have severity=`normal'? This is a BAD bug(s). It makes many html pages unviewable. Styling using stylesheets instead of tables etc is ruled out (contrary to W3c). And for xml only stylesheets are possible... I too have this bug. (M13, 27th Jan build, i686 linux on a pc) Hope you can fix it soon. --And THANKS!! for all your work on mozilla!
beard, you fixed this before, so i'm assigning it to you.
Assignee: pavlov → beard
Severity: normal → critical
Status: REOPENED → NEW
Keywords: beta1
Whiteboard: [PDT+] 01/16/00 → [PDT+] was fixed with viewmanager2
Target Milestone: M13 → M14
Blocks: 25824
*** Bug 21664 has been marked as a duplicate of this bug. ***
This requires a slight rewrite of the classic view manager to get this working correctly.
Status: NEW → ASSIGNED
No longer blocks: 25824
*** Bug 25782 has been marked as a duplicate of this bug. ***
Can someone who's been working on this check bugs 23626 and 25227 (list boxes cause gray stripe) to see if they are duplicates of this one?
*** Bug 23626 has been marked as a duplicate of this bug. ***
When this bug will be fixed? I can't use mozilla for common browsing due to this bug - it shows up on every third page which I visit.
*** Bug 22936 has been marked as a duplicate of this bug. ***
*** Bug 22936 has been marked as a duplicate of this bug. ***
*** Bug 22936 has been marked as a duplicate of this bug. ***
Putting dogfood in the keyword field.
*** Bug 26285 has been marked as a duplicate of this bug. ***
Adding syd and myself to cc list.
What horrible, degrading act do I have to perform and who do I have to do it to in order to get this bug fixed?
heh, we're having a meeting in a minute.
*** Bug 25492 has been marked as a duplicate of this bug. ***
*** Bug 18224 has been marked as a duplicate of this bug. ***
Ok.I verified that it works for me too using viewmanager2. Any ideas when it is gonna be turned on by default? This bug has really annoyed me for a long time.
Summary: [DOGFOOD][PP]absolute position elements repaint wrong w/GFX scrollbars on → absolute position elements repaint wrong w/GFX scrollbars on
*** Bug 24553 has been marked as a duplicate of this bug. ***
There's now a pref checked-in to turn this on by default. Warning, it doesn't handle opacity yet (coming soon).
*** Bug 27215 has been marked as a duplicate of this bug. ***
*** Bug 27515 has been marked as a duplicate of this bug. ***
*** Bug 27657 has been marked as a duplicate of this bug. ***
*** Bug 27562 has been marked as a duplicate of this bug. ***
*** Bug 27341 has been marked as a duplicate of this bug. ***
*** Bug 24141 has been marked as a duplicate of this bug. ***
*** Bug 27637 has been marked as a duplicate of this bug. ***
*** Bug 22573 has been marked as a duplicate of this bug. ***
*** Bug 27228 has been marked as a duplicate of this bug. ***
*** Bug 27443 has been marked as a duplicate of this bug. ***
*** Bug 25409 has been marked as a duplicate of this bug. ***
*** Bug 27516 has been marked as a duplicate of this bug. ***
*** Bug 23120 has been marked as a duplicate of this bug. ***
*** Bug 25672 has been marked as a duplicate of this bug. ***
Seeing this on cnn.com and pathfinder.com (in linux, build 2000021510), whcih, ironically enough, are two of the "verification" sites listed. This also appears on virtually any large/long page with a lot of tables if you scroll up and down fast enough.
This is fixed by nsViewManager2, which will be turned on by default for beta1.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
*** Bug 22907 has been marked as a duplicate of this bug. ***
*** Bug 28061 has been marked as a duplicate of this bug. ***
*** Bug 28613 has been marked as a duplicate of this bug. ***
Verified with todays build on linux, thinks look great now!
Status: RESOLVED → VERIFIED
*** Bug 28620 has been marked as a duplicate of this bug. ***
*** Bug 28620 has been marked as a duplicate of this bug. ***
*** Bug 29587 has been marked as a duplicate of this bug. ***
*** Bug 32405 has been marked as a duplicate of this bug. ***
For those who don't know already, this bug has come back, and has been reincarnated as bug 37360 (which is nsbeta2+).
No longer blocks: 21564
No longer blocks: 22176
No longer blocks: 24206
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: