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: InRadar, NeedsReduction
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

Description Ahmad Saleem 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!
Comment 1 Radar WebKit Bug Importer 2023-11-09 07:42:58 PST
<rdar://problem/118177925>
Comment 2 zalan 2023-11-14 17:00:11 PST
Created attachment 468597 [details]
test reduction
Comment 3 zalan 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)
Comment 4 zalan 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?
Comment 5 Karl Dubost 2023-11-20 22:14:21 PST
Created attachment 468700 [details]
rendering in safari, firefox, chrome

Really broken in Firefox.
Comment 6 Karl Dubost 2023-11-20 22:15:25 PST
Created attachment 468701 [details]
testcase rendering in safari, firefox, chrome
Comment 7 Karl Dubost 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.