[Webkit-unassigned] [Bug 85856] [Chromium] Cannot show the left side of an RTL element with a vertical scrollbar
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri May 25 03:04:10 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=85856
--- Comment #10 from Hironori Bono <hbono at chromium.org> 2012-05-25 03:03:14 PST ---
(From update of attachment 143519)
View in context: https://bugs.webkit.org/attachment.cgi?id=143519&action=review
>> Source/WebCore/rendering/RenderBlock.cpp:2869
>> + scrolledOffset.move(clientLeft(), 0);
>
> It looks like if we move by clientLeft, we also move by the borderLeft. Can you add a test with a left border and make sure it works?
D'oh. This is totally my mistake. Thanks for pointing it out.
>> Source/WebCore/rendering/RenderBox.h:200
>> + LayoutUnit clientLeft() const { return borderLeft() + (style()->shouldPlaceBlockDirectionScrollbarOnLogicalLeft() ? verticalScrollbarWidth() : 0); }
>
> Is this correct in vertical writing mode? The style parameter you check is for logical left, but clientLift is not logical left. Can you add a test for vertical writing mode? You might want to add the method clientLogicalStart to RenderBox.
Yes, shouldPlaceBlockDirectionScrollbarOnLogicalLeft() becomes false in the vertical-writing mode: <http://code.google.com/searchframe#OAMlx_jo-ck/src/third_party/WebKit/Source/WebCore/rendering/style/RenderStyle.h&exact_package=chromium&l=987>. (Nevertheless, it is totally my mistake to use clientLeft() in RenderBlock::paintObject(), though.)
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list