Skip to content

Commit

Permalink
[NFC] CSS highlight pseudos: tidy up spec issues that were resolved
Browse files Browse the repository at this point in the history
This patch removes comments referring to spec issues that have now
been resolved or reached a clear consensus with the editors.

Bug: 1024156, 1147859
Change-Id: I954164f02b5392d4a0b18f5c6614c334aee6332e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3610048
Reviewed-by: Manuel Rego <[email protected]>
Commit-Queue: Delan Azabani <[email protected]>
Cr-Commit-Position: refs/heads/main@{#996811}
  • Loading branch information
delan authored and chromium-wpt-export-bot committed Apr 27, 2022
1 parent a4b4e67 commit 71f67d1
Show file tree
Hide file tree
Showing 6 changed files with 2 additions and 8 deletions.
3 changes: 1 addition & 2 deletions css/css-pseudo/highlight-painting-003.html
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,7 @@
Decoration paints are ordered by text-decoration-line
(underline, overline, text, line-through), then by highlight
overlay (originating, ::grammar-error, ::spelling-error,
::target-text, ::selection). See options (d, d, d) in
csswg-drafts#6022 for more details.
::target-text, ::selection).
*/
* {
text-decoration-skip-ink: none;
Expand Down
3 changes: 1 addition & 2 deletions css/css-pseudo/highlight-painting-004.html
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@
Decoration paints are ordered by text-decoration-line
(underline, overline, text, line-through), then by highlight
overlay (originating, ::grammar-error, ::spelling-error,
::target-text, ::selection). See options (d, d, d) in
csswg-drafts#6022 for more details.
::target-text, ::selection).
*/
* {
text-decoration-skip-ink: none;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<!doctype html>
<meta charset="utf-8">
<!-- tentative due to unresolved spec issue: see question 3 in <https://github.com/w3c/csswg-drafts/issues/6386> -->
<title>CSS Pseudo-Elements Test: paired cascade: rule includes highlight pseudos other than ::selection</title>
<link rel="author" name="Delan Azabani" href="mailto:[email protected]">
<link rel="help" href="https://drafts.csswg.org/css-pseudo-4/#highlight-cascade">
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<!doctype html>
<meta charset="utf-8">
<!-- tentative due to unresolved spec issue: see question 6 in <https://github.com/w3c/csswg-drafts/issues/6386> -->
<title>CSS Pseudo-Elements Test: paired cascade: UA default highlight colors are not used when highlight pseudo has declared color value of unset</title>
<link rel="author" name="Delan Azabani" href="mailto:[email protected]">
<link rel="help" href="https://drafts.csswg.org/css-pseudo-4/#highlight-cascade">
Expand Down
1 change: 0 additions & 1 deletion css/css-pseudo/textpath-selection-011-ref.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
/*
color suppresses UA default background-color,
but fill and stroke do not, so set explicitly
https://github.com/w3c/csswg-drafts/issues/6386
*/
fill: green;
/*
Expand Down
1 change: 0 additions & 1 deletion css/css-pseudo/textpath-selection-011.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
/*
color suppresses UA default background-color,
but fill and stroke do not, so set explicitly
https://github.com/w3c/csswg-drafts/issues/6386
*/
fill: green;
/*
Expand Down

0 comments on commit 71f67d1

Please sign in to comment.