Bug 268269

Summary: Explore the usage of 'text-track.css' and 'mediaControls' as UA Stylesheet rule
Product: WebKit Reporter: Ahmad Saleem <ahmad.saleem792>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: annevk, destra, eric.carlson, jean-yves.avenard, jer.noble, ntim, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 268296    
Bug Blocks:    
Attachments:
Description Flags
Just a snapshot - Reference Image none

Description Ahmad Saleem 2024-01-28 20:14:05 PST
Hi Team,

I started noticing that for media controls, we have three UA stylesheets:

1) mediaControls.css
2) captions.css
3) text-track.css

__

I'll start with simple one - 'captions.css':

Only these exist in the file (which is not in either of others):

video::cue(:future) > *

video::cue(*) {

____

Then in text-track.css:

We have following to cover 'captions.css' one:

::cue(:future) 

and have two '::-webkit-media-text-track-display' (in other files as well but also twice in same and conflict with each other -> e.g., white-space: pre-wrap; and white-space: pre-line;

etc.

__

and mediaControls.css is the biggest.

____

I think we can get rid of `captions.css` while we can trim and clean-up `text-track.css` while make `mediaControls.css` as defacto standard.

Just raising to get input and also if someone want to tackle it.

Thanks!
Comment 1 Ahmad Saleem 2024-01-28 20:21:07 PST
Created attachment 469587 [details]
Just a snapshot - Reference Image

Diff for video::-webkit-media-text-track-container between:

> mediaControls.css
> text-tracks.css
Comment 2 Radar WebKit Bug Importer 2024-02-04 20:15:19 PST
<rdar://problem/122289181>
Comment 3 Ahmad Saleem 2024-02-20 09:54:17 PST
`captions.css` is gone since it was accidentally added.