[webkit-changes] [WebKit/WebKit] bb909a: [Cocoa] ::cue background-color not changed unless ...

Jer Noble noreply at github.com
Tue Jul 11 17:42:53 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: bb909a8331522160a271cc9943fa804d4b1c8a5c
      https://github.com/WebKit/WebKit/commit/bb909a8331522160a271cc9943fa804d4b1c8a5c
  Author: Jer Noble <jer.noble at apple.com>
  Date:   2023-07-11 (Tue, 11 Jul 2023)

  Changed paths:
    A LayoutTests/media/track/track-user-stylesheet-override-expected.txt
    A LayoutTests/media/track/track-user-stylesheet-override.html
    M Source/WebCore/Modules/modern-media-controls/controls/text-tracks.css

  Log Message:
  -----------
  [Cocoa] ::cue background-color not changed unless "allow video to override" unchecked
https://bugs.webkit.org/show_bug.cgi?id=259121
rdar://112100260

Reviewed by Andy Estes.

The `text-tracks.css` UA style sheet sets default styles for VTT cues by setting
a style on `video::cue`. However that has a higher specificity than the styles generated
by the platform, which use `::cue`. This means, in effect, that the system provided
styles (as specified by the user) are ignored unless they are accompanied by a '!important'
rule, which means those rules also cannot be overridden by page provided styles.

Move from `video::cue` to `::cue` in the UA provided styles, allowing both the platform
and the page to override them.

* LayoutTests/media/track/track-user-stylesheet-override-expected.txt: Added.
* LayoutTests/media/track/track-user-stylesheet-override.html: Added.
* Source/WebCore/Modules/modern-media-controls/controls/text-tracks.css:
(::cue):
(::-webkit-media-text-track-display):
(::-webkit-media-text-track-display-backdrop):
(::cue(:future)):
(video::cue): Deleted.
(video::-webkit-media-text-track-display): Deleted.
(video::-webkit-media-text-track-display-backdrop): Deleted.
(video::cue(:future)): Deleted.

Canonical link: https://commits.webkit.org/265975@main




More information about the webkit-changes mailing list