WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
267615
`audio` element has two UA Stylesheet rules (html.css & mediaControls.css)
https://bugs.webkit.org/show_bug.cgi?id=267615
Summary
`audio` element has two UA Stylesheet rules (html.css & mediaControls.css)
Ahmad Saleem
Reported
2024-01-16 17:42:49 PST
Hi Team, I noted that we have two `audio` element UA stylesheet rules: html.css:
https://github.com/WebKit/WebKit/blob/8948469628f6e2c8cd44d680b2f194371e9b3f97/Source/WebCore/css/html.css#L174
audio { width: 200px; height: 25px; } mediaControls.css
https://github.com/WebKit/WebKit/blob/8948469628f6e2c8cd44d680b2f194371e9b3f97/Source/WebCore/css/mediaControls.css#L31
audio { width: 200px; height: 16px; } __ Don't know which one to delete. Only difference is 'height'. Any input? Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2024-01-16 17:44:59 PST
NOTE - Web spec don't have any specified width and height for audio elements.
https://html.spec.whatwg.org/#embedded-content-rendering-rules
Ahmad Saleem
Comment 2
2024-01-16 18:06:27 PST
PR -
https://github.com/WebKit/WebKit/pull/22846
Using 'html.css' as single source of truth.
EWS
Comment 3
2024-01-16 22:13:09 PST
Committed
273114@main
(f2379cbda4fa): <
https://commits.webkit.org/273114@main
> Reviewed commits have been landed. Closing PR #22846 and removing active labels.
Radar WebKit Bug Importer
Comment 4
2024-01-16 22:14:16 PST
<
rdar://problem/121095621
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug