[Webkit-unassigned] [Bug 54623] RTL web content should have left-hand scrollbar.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=54623





--- Comment #41 from Ryosuke Niwa <rniwa at webkit.org>  2012-01-06 10:40:16 PST ---
(From update of attachment 121427)
View in context: https://bugs.webkit.org/attachment.cgi?id=121427&action=review

Great! New patch looks much cleaner.

> Source/WebCore/rendering/RenderLayer.cpp:1952
> +LayoutUnit RenderLayer::rtlAwareVerticalScrollbarX(int minX, int maxX) const

We normally use the word "start" to mean left for LTR and right for RTL so how about verticalScrollbarStart?

> Source/WebCore/rendering/RenderLayer.cpp:1962
> +LayoutUnit RenderLayer::rtlAwareHorizontalScrollbarX(int minX) const

horizontalScrollbarStart?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.