Bug 16426

Summary: Divs with overflow:auto: scrollbars not correctly updated when contents change
Product: WebKit Reporter: Khoo Yit Phang <khooyp>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mitz
Priority: P2 Keywords: HasReduction
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Page showing problem with overflow:auto divs' scrollbars
none
Update the NSScroller's value when the proportions change darin: review+

Khoo Yit Phang
Reported 2007-12-13 13:04:26 PST
I have a page set up with a overflow:auto div element. When the contents of the div change (e.g. some children set to display:none), the scrollbar on the div element doesn't seem to be updated correctly. If the user then clicks on the scrollbar or attempts to scroll, the contents will jump to wherever the scrollbar was incorrectly set to. This doesn't seem to occur on the main scrollbars, just on elements with overflow:auto set. I'll attach a test page that shows this effect.
Attachments
Page showing problem with overflow:auto divs' scrollbars (1.57 KB, application/xhtml+xml)
2007-12-13 13:05 PST, Khoo Yit Phang
no flags
Update the NSScroller's value when the proportions change (33.74 KB, patch)
2007-12-16 01:45 PST, mitz
darin: review+
Khoo Yit Phang
Comment 1 2007-12-13 13:05:38 PST
Created attachment 17878 [details] Page showing problem with overflow:auto divs' scrollbars
David Kilzer (:ddkilzer)
Comment 2 2007-12-15 05:59:22 PST
Confirmed behavior with a local debug build of WebKit r28711 with Safari 3.0.4 (523.12) on Mac OS X 10.4.11 (8S165).
mitz
Comment 3 2007-12-15 15:10:35 PST
I think this affects the Web Inspector.
mitz
Comment 4 2007-12-16 01:45:34 PST
Created attachment 17927 [details] Update the NSScroller's value when the proportions change Also removes a repaint() that seems unnecessary, especially in light of the fact that the horizontal scrollbar has never been repainted: the code that was supposed to do it was accidently #ifdef'ed out since day one and ultimately removed: <http://trac.webkit.org/projects/webkit/changeset/6395>.
Darin Adler
Comment 5 2007-12-16 10:17:39 PST
Comment on attachment 17927 [details] Update the NSScroller's value when the proportions change r=me
mitz
Comment 6 2007-12-16 10:41:34 PST
Note You need to log in before you can comment on or make changes to this bug.