[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


Hironori Bono <hbono at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #122368|0                           |1
        is obsolete|                            |
 Attachment #122368|review?                     |
               Flag|                            |
 Attachment #125303|                            |review?
               Flag|                            |




--- Comment #48 from Hironori Bono <hbono at chromium.org>  2012-02-03 03:32:43 PST ---
Created an attachment (id=125303)
 --> (https://bugs.webkit.org/attachment.cgi?id=125303&action=review)
Patch v10 (Applied comments)

Greetings Niwa-san,

Many thanks for your comments.

(In reply to comment #47)
> Sorry, that comment seems bogus. What I really meant is move "if (positionedObject->style()->position() != FixedPosition)" outside of if-defs to be shared. Then, you only need to wrap two lines inside if-defs (x -= verticalScrollbarWidth(); and if before that).

Done. Thanks for your correction.

> Hm... how about shouldPlaceBlockDirectionScrollbarOnLogicalLeft() then ?

Done. I should have describe this issue when I noticed it.

Regards,

Hironori Bono

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