-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
www.google.com – Time is cropped on “Popular times” graphic #7356
Comments
The time is defined by <div style="" class="i_NOYJDb5a6c-17AgC1q3TaA _epj">
<div>
<div class="_ipj">
<div>6a</div>
</div>
</div>
<div></div>
<div></div>
<div>
<div class="_ipj">
<div>9a</div>
</div>
</div>
<div></div>
<div></div>
<!-- cut for brevity -->
</div> but there is a block for the full graphic. which is defined by <div class="_ppj i_NOYJDb5a6c-JjrbqILQZ8E"
data-ved="0ahUKEwiTlabcm8nUAhWBUrwKHVXxCE8QoWEIYTAN"
style="height: 135px;">
<!-- cut -->
</div> and has a fixed height. if I change the height to be ._ipj {
height: 18px;
position: relative;
-webkit-user-select: none;
white-space: nowrap;
} It's why the pink doesn't go full in the red outline and a combination of line-height. all in all the design is not very robust depending on font-size etc. setting as contactready but not really made for contact as of now. |
@karlcow what would the suggested fix be in this case? It seems like an interop issue around font line-height? Or did I misunderstand. |
|
While the symptom here seems slightly different (a There is also another case of usage of "Roboto-Regular" instead of "Roboto", which only further complicates the issue as Firefox mobile is likely selecting Clear Sans instead of Roboto, which is being investigated by Blink here. |
URL: https://google.com
Browser / Version: Firefox Mobile Nightly 55.0a1 (2017-06-09) - Chome UA
Operating System: LG G5 (Android 7.0) - Resolution 1440 x 2560 pixels (~554 ppi pixel density)
Problem type: Text is not visible
Steps to Reproduce
Expected Behavior:
Time is fully displayed.
Actual Behavior:
Time is cropped.
Note:
Watchers:
@softvision-sergiulogigan
@softvision-oana-arbuzov
sv; gs
From webcompat.com with ❤️
The text was updated successfully, but these errors were encountered: