Regression: Spacing is not proper for radio buttons in blocked pop-up bubble.
Reported by
aiman.an...@etouch.net,
Jan 9 2018
|
|||
Issue descriptionChrome Version: 65.0.3315.4 (Official Build) f9677678abcb81cc8c22c4a489c5d7170e87cc9a-refs/branch-heads/3315@{#5} (32/64-bit) OS: Win(7,8,8.1,10), Mac(10,12,6, 10.13.1, 10.13.3) and Linux(14.04 LTS). Pre-Condition: Enable Force UI Direction(Right-to-left) flag. Test URL: permission.site What steps will reproduce the problem? 1. Launch chrome, navigate to the above URL and click on Popup (delayed 2 seconds). 2. Click on pop-up blocked icon in omnibox and observe. Actual: Spacing is not proper for radio buttons in blocked pop-up bubble. Expected: Spacing for radio buttons should be proper in blocked pop-up bubble. This is a regression issue, broken in M-65 series, Using the per-revision bisect providing the bisect results, Good Build:65.0.3286.0(Revision:526410) Bad Build:65.0.3287.0(Revision:526575) You are probably looking for a change made after 522118 (known good), but no later than 522119 (first known bad). CHANGE-LOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/3425c94e632ca511ad9dcf7c1c4e3b49fcfcf0dc..12d3545cdca172bf6e8f1fb7e831a2d12c916f26 Suspect:https://chromium.googlesource.com/chromium/src/+/12d3545cdca172bf6e8f1fb7e831a2d12c916f26 ellyjones@:Kindly help to reassign, if your changes are not related to this issue.
,
Jan 9 2018
Thanks for your report. This is interesting. I reproduced that behavior in an English locale with --force-ui-direction=rtl, but the same problem does not appear in a real RTL locale, such as Hebrew (see the attached screenshot). While the English text layout does look wrong, it's because the controls are (rightly) assuming an RTL layout is in use but the text is still using LTR, so the wrong appearance won't happen in practice for users unless they too are using --force-ui-direction=rtl. In conclusion, WontFix - both pieces are working as intended, and this is a logical result of using an LTR script in an RTL layout. You can see the same issue in other parts of the browser with --force-ui-direction=rtl.
,
Feb 2 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by ranjitkan@chromium.org
, Jan 9 2018