Bug 264474

Summary:

'Play' button on NASA+ Service is too large

Product:

WebKit

Reporter:

Ahmad Saleem <ahmad.saleem792>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

NEW ---

Severity:

Normal

CC:

bfulgham, fantasai.bugs, karlcow, simon.fraser, vitor.roriz, webkit-bug-importer, zalan

Priority:

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.