Bug 264474

Summary: 'Play' button on NASA+ Service is too large
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: bfulgham, fantasai.bugs, karlcow, simon.fraser, vitor.roriz, webkit-bug-importer, zalan
Priority: P2 Keywords: BrowserCompat, InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Broken 'red' play button
none
test reduction
none
rendering in safari, firefox, chrome
none
testcase rendering in safari, firefox, chrome none

Ahmad Saleem
Reported 2023-11-09 04:27:09 PST
Created attachment 468532 [details] Broken 'red' play button Hi Team, While trying to explore new Nasa+ service, while trying to check whether the service work properly or not, I noticed a bug. URL: https://plus.nasa.gov/video/we-make-history-here/ ^ Please refer to attach screenshot and it is broken on Firefox Nightly 121 as well. (So it indicates more of lack of testing done on other browsers from website developer side). Just wanted to raise so we can fix it. Thanks!
Attachments
Broken 'red' play button (588.46 KB, image/png)
2023-11-09 04:27 PST, Ahmad Saleem 		no flags
Details
test reduction (209 bytes, text/html)
2023-11-14 17:00 PST, alan 		no flags
Details
rendering in safari, firefox, chrome (343.70 KB, image/png)
2023-11-20 22:14 PST, Karl Dubost 		no flags
Details
testcase rendering in safari, firefox, chrome (166.20 KB, image/png)
2023-11-20 22:15 PST, Karl Dubost 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-11-09 07:42:58 PST
<rdar://problem/118177925>
alan
Comment 2 2023-11-14 17:00:11 PST
Created attachment 468597 [details] test reduction
alan
Comment 3 2023-11-14 19:56:14 PST
This is caused by an intentional webkit specific behavior of not letting the computed font size go below 9px (after applying the zoom value), which means that em values are resolved against this capped font-size value resulting inflated height/width values (thus the "large" play button)
alan
Comment 4 2023-11-15 11:41:55 PST
(In reply to zalan from comment #3) > This is caused by an intentional webkit specific behavior of not letting the > computed font size go below 9px (after applying the zoom value), which means > that em values are resolved against this capped font-size value resulting > inflated height/width values (thus the "large" play button) It looks like this should be driven by Safari's setting [Advanced -> "Never use font sizes smaller than], but even when that option is unchecked (default behavior) we apply 9px as the lowest allowed size value for fonts. Maybe we are missing an "is this enabled" preference key?
Karl Dubost
Comment 5 2023-11-20 22:14:21 PST
Created attachment 468700 [details] rendering in safari, firefox, chrome Really broken in Firefox.
Karl Dubost
Comment 6 2023-11-20 22:15:25 PST
Created attachment 468701 [details] testcase rendering in safari, firefox, chrome
Karl Dubost
Comment 7 2023-11-21 00:06:51 PST
To better understand the test changing the font-size on body to 100px makes the test work for both Safari and Chrome. Firefox (recent) implementation of zoom is not working at all.
Note You need to log in before you can comment on or make changes to this bug.