RTL scrollbar is misplaced on Mac |
||||||||||||
Issue descriptionVersion: 51, 54 OS: OS X 10.11.5 What steps will reproduce the problem? data:text/html;charset=utf8,<div dir="rtl" style="height: 100px; width: 300px; background: aliceblue; overflow: scroll"><div style="height: 600px;"> Scroll on box What is the expected output? Scroll shouldn't have unnecessary right padding What do you see instead? It has right padding that you can't see on Firefox for example.
,
Jul 11 2016
Position of scrollbars is solely a layout thing; removing Blink>Scroll tag.
,
Jul 12 2016
Szager loves RTL scrollbar bugs. True story.
,
Jul 12 2016
I tried this on a Mac running Chrome version 51.0.2704.106, and it looks fine to me: http://imgur.com/Kur031C Can you post a screenshot showing the bug?
,
Jul 12 2016
,
Jul 12 2016
Weird, why your scroll differs that much from mine? Using latest Canary without any experimental flag (/Applications/Google Chrome Canary.app/Contents/MacOS/Google Chrome Canary --flag-switches-begin --flag-switches-end) I see the attachment
,
Jul 12 2016
Expected result (but on LTR attached). I remember on other OSes there was a flag called overlay scrollbar or something and it seems you've enabled it by default on Mac at least as I can't see any option on about:flags to disable it even.
,
Jul 13 2016
,
Aug 12 2016
Increasing its priority a bit.
,
Nov 1 2016
ebrahim@: does your Mac have a Retina screen? The presence of a Retina can affect compositing decisions and could help narrow down where the problem lies.
,
Nov 1 2016
It is an old one without a retina screen
,
Nov 1 2016
Got it, I suspect szager@ had a retina screen. Could you try starting Chrome with the --enable-prefer-compositing-to-lcd-text flag and see if the issue still reproduces?
,
Nov 1 2016
Alternatively, you can force the scroller to be composited by adding "transform: translateZ(0)" to its style.
,
Nov 1 2016
It makes it more funny but no different. Actually I think this is a place someone should have look https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/scroll/ScrollbarTheme.cpp?sq=package:chromium&dr=CSs&l=323 "thumbPos + thumbRect.width() / 2,"
,
Nov 1 2016
Interesting. Sahel is currently looking at RTL scrollbar positioning for ChromeOS but we can take a look at this after that.
,
Nov 2 2016
c#14 is perhaps less related, interesting that overlay scrollbar that enables similar scrollbar on the rest of the platform doesn't have the issue.
,
Nov 8 2016
,
Nov 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/94a2767135371369605eda754487ea11ff7213b2 commit 94a2767135371369605eda754487ea11ff7213b2 Author: sahel <sahel@chromium.org> Date: Wed Nov 23 20:30:54 2016 RTL overlay scrollbar on Mac is left aligned. BUG= 626919 Review-Url: https://codereview.chromium.org/2512033002 Cr-Commit-Position: refs/heads/master@{#434239} [modify] https://crrev.com/94a2767135371369605eda754487ea11ff7213b2/third_party/WebKit/Source/platform/scroll/ScrollbarThemeMac.mm
,
Nov 23 2016
Before and after screenshots are attached. |
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by tkent@chromium.org
, Jul 10 2016Components: -Blink Blink>Layout Blink>Scroll