Summary: | Vertical scroll bars do not appear or do not scroll completely. | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | david kirk <davak> | ||||||||||||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | jonathanjohnsson, mitz | ||||||||||||||
Priority: | P2 | Keywords: | HasReduction | ||||||||||||||
Version: | 417.x | ||||||||||||||||
Hardware: | PC | ||||||||||||||||
OS: | OS X 10.4 | ||||||||||||||||
URL: | http://www.tech-recipes.com | ||||||||||||||||
Attachments: |
|
Description
david kirk
2006-02-03 12:58:57 PST
Created attachment 6225 [details]
stylesheet showing bug
Created attachment 6226 [details]
html of the bug
Created attachment 6227 [details]
html code of the bug
In order to attempt to help, I have created a barebones html structure documenting this bug: http://www.bronch.com/downloads/webkit_bug/index.html Created attachment 6245 [details] Testcase Attaching David Kirk's reduction from http://www.bronch.com/downloads/webkit_bug/index.html This can probably be further reduced, so I'm leaving the NeedsReduction keyword for now. Created attachment 6333 [details] Testcase reduction (In reply to comment #5) > Created an attachment (id=6245) [edit] > Testcase > > Attaching David Kirk's reduction from > http://www.bronch.com/downloads/webkit_bug/index.html > > This can probably be further reduced, so I'm leaving the NeedsReduction keyword > for now. > I tried to reduce it as much as possible, without removing the scrollbar issue. (Used the above testcase as base, hope that's ok.) The page looks the same in HTML 4.01 Strict. Removing any of the divs or the table, or hanging any of the divs' position values, makes the vertical scrollbar appear. My first try at this sort of thing. Couldn't connect to #webkit, to get to know how to do this the best/right way. Great reduction! Created attachment 6384 [details]
Implement {leftmost, rightmost, lowest}Position for RenderTableSection
The patch affects an existing test, fast/replaced/width100percent.html, and includes updated results. That test shows a repaint problem when scrolling, but that's not caused by the patch (if you make the window narrow enough and scroll it horizontally w/o the patch it also fails to repaint completely in TOT).
Comment on attachment 6384 [details]
Implement {leftmost, rightmost, lowest}Position for RenderTableSection
Looks great to me, but this is the kind of patch Hyatt should review.
Comment on attachment 6384 [details]
Implement {leftmost, rightmost, lowest}Position for RenderTableSection
r=me
|