Issue metadata
Sign in to add a comment
|
Regression : ‘New tab’ button overlaps with ‘Minimise’ and ‘Full screen’ button in RTL mode.
Reported by
avsha...@etouch.net,
Dec 6 2017
|
||||||||||||||||||||||||
Issue descriptionChrome Version : 63.0.3239.84 (Official Build) 8f51ed0e633e109109762a3deb18a50e8c138819-refs/branch-heads/3239@{#643} 64-bit OS : Mac(10.12.6) (10.13.2) Precondition : Please enable ‘Force UI direction’ and ‘Enable RTL’ flags from chrome://flags. What steps will reproduce the problem? 1. Launch chrome, enable ‘Force UI direction’ and ‘Enable RTL’ flags and restart the browser. 2. Open 7 to 8 NTP and observe the browser’s ‘Minimise’ and ‘Full screen’ button. (Kindly review an attached screen cast) Actual Result : ‘New tab’ button overlaps with ‘Minimise’ and ‘Full screen’ button in RTL mode. Expected Result : ‘New tab’ button should not overlap with ‘Minimise’ and ‘Full screen’ button in RTL mode. This is a regression issue broken in ‘M-62’ and using the per-revision bisect providing the bisect results, Good build : 62.0.3180.0 (Revision : 492770) Bad build : 62.0.3181.0 (Revision : 493197) You are probably looking for a change made after 492971 (known good), but no later than 492972 (first known bad). CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/d5ea6ad9b7b5af521c634c6461227c619b6728a1..1f71c6ec3d33ac53f8e2474fbfd9e6396f731008 Suspect : https://chromium.googlesource.com/chromium/src/+/1f71c6ec3d33ac53f8e2474fbfd9e6396f731008 @sdy : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner. Note : This is Mac OS specific issue and the same is not reproducible on Windows(7,8,10) and Linux(14.04 LTS) OS
,
Dec 6 2017
I reported this as well ( Issue 789350 ). The problem is just enabling those flags isn't enough to run Chrome in RTL mode. You have to also supply -NSForceRightToLeftWritingDirection YES -AppleTextDirection YES to the command line. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by msrchandra@chromium.org
, Dec 6 2017