Bug 264474 - 'Play' button on NASA+ Service is too large
Summary: 'Play' button on NASA+ Service is too large
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, NeedsReduction
Depends on:
Blocks:
 
Reported: 2023-11-09 04:27 PST by Ahmad Saleem
Modified: 2023-11-21 00:06 PST (History)
7 users (show)

See Also:


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, zalan
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

Note You need to log in before you can comment on or make changes to this bug.
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.