Font icons are sometimes cut off horizontally when the window viewport height is below a certain threshold
Reported by
liug.2...@gmail.com,
Nov 10 2016
|
|||||||||||
Issue description
Chrome Version : 54.0.2840.87
URLs (if applicable) : See attachment
Other browsers tested:
Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
Safari: OK
Firefox: OK
IE: OK
Opera: OK
What environment will reproduce the problem?
* MacOS, retina display (tested on 13" MB with Retina and 15" MBP with Retina)
What steps will reproduce the problem?
(1) Unzip the attached HTML (chrome-bug.zip)
(2) Open the included HTML file in a new Chrome window on the retina display
(3) Resize the browser window so that the height of the viewport is less than 550px
What is the expected result?
The font icon is not clipped in any way. See rendering on Firefox, Safari, Opera, IE.
What happens instead?
The font icon is clipped horizontally, depending on the browser viewport height.
Please provide any additional information below. Attach a screenshot if
possible.
Attached a minimal case, and a screenshot.
,
Nov 11 2016
My apologies, here you go.
,
Nov 19 2016
Thank you for providing more feedback. Adding requester "hdodda@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 6 2016
Tested on mac os 10.11.6 , 10.12.1 retina using chrome stable M55 #55.0.2883.75 and issue is not reproduced. Attached screenshot for reference. @liug.2012 -- Could you please check in latest stable and update us your observations. Thanks!
,
Dec 7 2016
,
Dec 14 2016
Adding 'Needs-Milestone' label, TE will check the issue and update the bug with comments & tag with respective Mstone once the issue gets confirmed.
,
Dec 15 2016
@hdodda I am still able to reproduce the issue. I'm using mac os 10.11.6, Chrome 55.0.2883.95 (64-bit).
,
Dec 15 2016
For reference, my browser viewport height is 453px. Viewport width seems to have no effect on reproducing or not reproducing the bug.
,
Dec 23 2016
Thank you for providing more feedback. Adding requester "hdodda@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 27 2016
Tested on mac os 10.12.2 using chrome latest stable M55 #55.0.2883.95 and issue is not reproduced in different system displays. @could someone from webgonts team look into this. Thanks!
,
Jan 6 2017
This reproduced on my non-retina MacBook Air, by zooming the page (command-+) into 200%. This reproduced on 55.0.2883.95 stable, but not on 57.0.2964.0 canary. Probably it's already fixed at somewhere between M55 and M57? Adding Needs-Bisect to identify at what point it was fixed.
,
Jan 10 2017
Tested on Mac os 10.12.2 using chrome M55 #55.0.2883.95 stable and working fine on beta 56.0.2924.51 and chrome canary M57 #57.0.2976.0 . Steps to reproduce: Opened the given html file in comment #2 , i.e., index.html and zoomed the page to 200% .. Observed that the GroupON fonts are chopped. Reverse Bisect Info : Good Build : 56.0.2894.0 (Revision : 425529) Bad Build : 56.0.2891.0 (Revision : 425838) Unable to provide the tool bisect info as all the builds invoked were good builds. Hence providing manual chnage log. Manual change log : https://chromium.googlesource.com/chromium/src/+log/56.0.2891.0..56.0.2894.0?pretty=fuller&n=10000 Suspecting the CL: Review URL: https://codereview.chromium.org/2422123002 @zakerinasab-- Could you please merge this to M55 if it is a valid candidate to merge. Thanks!
,
Jan 10 2017
I just checked this on Chrome Canary, 57.0.2977.0. The issue is not reproduced.
,
Jan 10 2017
@hdodda, I'm not sure if my CL has fixed the issue as my CL concerns the baseline for very small fonts before or after zooming (bugs.chromium.org/p/chromium/issues/detail?id=338908). I'm also not a committer. @eae can take another look and merge to M55 if necessary.
,
Jan 11 2017
Thanks hdodda@ for the bisect. I think merge is not needed, as M55 is stable channel now and this is a Pri-3 bug. Let me close this as fixed in M56. |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by hdodda@chromium.org
, Nov 11 2016Labels: Needs-Feedback