|Summary:||No vertical scrollbar after the CSS class change|
|Component:||Layout and Rendering||Assignee:||Shinichiro Hamaji <hamaji>|
|Severity:||Normal||CC:||eric, hamaji, hyatt, simon.fraser|
|Version:||528+ (Nightly build)|
Description Nickolay 2010-03-22 13:27:24 PDT
The vertical scrollbar do not appears after the CSS class which suppress them gets removed. Attaching the test case. It fails on latests Safari for Windows, and Chrome for Ubuntu. The case contains a long text with vertical scrollbar, initially suppressed by css class. After pressing 'Unmask' the css class is being removed, but no scrollbars appears. Regards, Nickolay
Comment 3 Shinichiro Hamaji 2010-03-29 10:23:49 PDT
A few info in addition to WebCore/ChangeLog in my patch: - This issue also happens with horizontal scrollbars - I guess this was introduced in http://trac.webkit.org/changeset/5085 (so I'm CCing dhyatt) - I've confirmed my testcase doesn't show scrollbars without this patch
Comment 4 Simon Fraser (smfr) 2010-03-29 10:47:21 PDT
Your testcase doesn't have to contain so much text. It could be just <div style="height: 1000px">
Comment 6 Shinichiro Hamaji 2010-03-29 11:07:13 PDT
Created attachment 51932 [details] Patch v2 - rebased
Comment 7 Shinichiro Hamaji 2010-03-29 11:09:07 PDT
(In reply to comment #4) > Your testcase doesn't have to contain so much text. It could be just <div > style="height: 1000px"> Thanks for the comment! I've fixed my test case (and rebased to avoid purple bot status).
Comment 9 Dave Hyatt 2010-03-29 11:11:11 PDT
Comment on attachment 51932 [details] Patch v2 - rebased r=me
Comment 10 Eric Seidel (no email) 2010-03-29 11:39:54 PDT
Attachment 51932 [details] was posted by a committer and has review+, assigning to Shinichiro Hamaji for commit.
Comment 11 Shinichiro Hamaji 2010-03-29 21:35:43 PDT
Thanks for your quick reviews!
Comment 12 Shinichiro Hamaji 2010-03-29 23:15:43 PDT
Committed r56764: <http://trac.webkit.org/changeset/56764>