CSS modules: separate out unsupported features #40576

estelle · 2025-07-31T00:07:21Z

took a look at documented features that have no support

github-actions · 2025-07-31T00:09:27Z

Preview URLs (33 pages)

Flaws (6)

Note! 29 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/CSS/@color-profile
Title: @color-profile
Flaw count: 1

broken_links:
- Can't resolve /en-US/docs/Web/CSS/declaration-list

URL: /en-US/docs/Web/CSS/CSS_conditional_rules
Title: CSS conditional rules
Flaw count: 1

macros:
- Macro produces link /en-US/docs/Web/CSS/at-rule which is a redirect

URL: /en-US/docs/Web/CSS/CSS_masking
Title: CSS masking
Flaw count: 1

macros:
- Macro produces link /en-US/docs/Web/CSS/url which is a redirect

URL: /en-US/docs/Web/CSS/image/image
Title: image()
Flaw count: 3

broken_links:
- Can't resolve /en-US/docs/Web/CSS/url-modifier
- Can't resolve /en-US/docs/Web/CSS/url-token
- Can't resolve /en-US/docs/Web/CSS/url-modifier

External URLs (1)

URL: /en-US/docs/Web/CSS/line-height-step
Title: line-height-step

https://drafts.csswg.org/css-rhythm/ (1 time) (Note! This may be a new URL 👀)

(comment last updated: 2025-08-04 11:45:21)

Josh-Cena

You are missing:

I think we should have a uniform template for noting non-support in module landing pages and other pages, and apply it to all other existing notes

chrisdavidmills

@estelle a few bits for you to consider, but nothing major. Nice work.

chrisdavidmills · 2025-08-03T15:23:20Z

files/en-us/web/css/css_fonts/index.md

-  - : Descriptor:
-    - {{cssxref("@font-feature-values/font-display", "font-display")}}
+
+The specification also defines the {{cssxref("@font-feature-values/font-display", "font-display")}} descriptor which is not yet supported by any browser.


Suggested change

The specification also defines the {{cssxref("@font-feature-values/font-display", "font-display")}} descriptor which is not yet supported by any browser.

The specification also defines the {{cssxref("@font-feature-values/font-display", "font-display")}} descriptor, which is not yet supported by any browser.

Also, I'd think about how this section is rendered and try to make it easier to follow. The "At-rule..." line that precedes this line is bulleted, but the other "At-rule..." lines aren't.

And the "The specification also defines..." paragraphs aren't indented, which makes it hard to figure out which At-rule section they relate to. Would it be easier to follow if you made the "At-rule..." lines into sub-headings, and/or made the "The specification also defines..." paragraphs into notes?

I changed all the pages to make them uniform. Basically:

The X module also introduces Y. Currently, no browsers support this/these feature(s).

it is possible that they are implemented behind a flag, so didn't want to use that lingo (pedantic).

chrisdavidmills · 2025-08-03T15:28:02Z

files/en-us/web/css/css_masking/index.md

@@ -36,13 +36,14 @@ While masking provides more control and options, clipping can perform better if
 - {{cssxref("mask-size")}}
 - {{cssxref("mask-type")}}
 - {{cssxref("mask-border")}} shorthand
- {{cssxref("mask-border-mode")}}
 - {{cssxref("mask-border-outset")}}
 - {{cssxref("mask-border-repeat")}}
 - {{cssxref("mask-border-slice")}}
 - {{cssxref("mask-border-source")}}
 - {{cssxref("mask-border-width")}}



Remove blank line?

chrisdavidmills · 2025-08-03T15:29:22Z

files/en-us/web/css/css_selectors/index.md

 - `~` ([Subsequent sibling combinator](/en-US/docs/Web/CSS/Subsequent-sibling_combinator))
 - " " ([Descendant combinator](/en-US/docs/Web/CSS/Descendant_combinator))
 - `|` ([Namespace separator](/en-US/docs/Web/CSS/Namespace_separator))
 - `,` ([Selector list](/en-US/docs/Web/CSS/Selector_list))

+> [!NOTE]
+> The CSS selectors module introduces the [column combinator](/en-US/docs/Web/CSS/Column_combinator) (`||`) that has not yet been implemented.


Suggested change

> The CSS selectors module introduces the [column combinator](/en-US/docs/Web/CSS/Column_combinator) (`||`) that has not yet been implemented.

> The CSS selectors module introduces the [column combinator](/en-US/docs/Web/CSS/Column_combinator) (`||`), which is not yet supported by any browser.

To make it consistent with the previous similar notes in the CSS fonts module page?

Also, this is a note, and they aren't. I think having them all as notes is a good idea.

They should definitely be consistent. Notes are to alert people to things, and stuff that is written but not implemented is more of an aside. i am going to de-note them in the module landing pages.

chrisdavidmills · 2025-08-03T15:31:56Z

files/en-us/web/css/css_selectors/index.md

@@ -98,6 +96,9 @@ Selectors, whether used in CSS or JavaScript, enable targeting HTML elements bas
 - [Type selectors](/en-US/docs/Web/CSS/Type_selectors)
 - [Universal selectors](/en-US/docs/Web/CSS/Universal_selectors)

+> [!NOTE]
+> The CSS selectors module introduces the {{CSSXref(":blank")}}, {{CSSXref(":current")}}, {{CSSXref(":local-link")}}, and {{CSSXref(":target-within")}} pseudo-classes, which have not yet been implemented.


Suggested change

> The CSS selectors module introduces the {{CSSXref(":blank")}}, {{CSSXref(":current")}}, {{CSSXref(":local-link")}}, and {{CSSXref(":target-within")}} pseudo-classes, which have not yet been implemented.

> The CSS selectors module introduces the {{CSSXref(":blank")}}, {{CSSXref(":current")}}, {{CSSXref(":local-link")}}, and {{CSSXref(":target-within")}} pseudo-classes, which are not yet supported by any browser.

Again, for consistency with previous notes?

agree. Going with "support" instead of "implement" for pedantic reasons.

chrisdavidmills · 2025-08-03T15:32:40Z

files/en-us/web/css/css_values_and_units/index.md

@@ -49,7 +49,7 @@ Every CSS declaration consists of a property/value pair. The value can take vari
 - {{cssxref("tan()")}}
 - {{cssxref("url_function", "url()")}}

-Additional functions, including `calc-mix()`, `crossorigin()`, `first-valid()`, `if()`, `integrity()`, `progress()`, `random()`, `random-item()`, `referrerpolicy()`, `src()`, `type()`, and `toggle()`, are defined in the specifications, but not yet implemented in browsers.
+The CSS values and units module introduces the `calc-mix()`, `crossorigin()`, `first-valid()`, `integrity()`, `progress()`, `random()`, `random-item()`, `referrerpolicy()`, `src()`, `type()`, and `toggle()` functions, which are not yet supported in any browser.


chrisdavidmills · 2025-08-03T15:32:47Z

files/en-us/web/css/css_values_and_units/index.md

@@ -86,6 +84,8 @@ Additional functions, including `calc-mix()`, `crossorigin()`, `first-valid()`,
 - [`<url-modifier>`](/en-US/docs/Web/CSS/url_function#url-modifier)
 - [`<view-timeline-name>`](/en-US/docs/Web/CSS/view-timeline-name)

+The CSS values and units module introduces the [`<frequency>`](/en-US/docs/Web/CSS/frequency) and [`<frequency-percentage>`](/en-US/docs/Web/CSS/frequency-percentage) data types, which are not yet supported in any browser.


chrisdavidmills · 2025-08-03T15:33:32Z

files/en-us/web/css/frequency-percentage/index.md

@@ -62,7 +62,7 @@ Where a `<frequency-percentage>` is specified as an allowable type, this means t

 ## Browser compatibility

-No browser currently supports this feature.
+Currently, no browsers support this feature.


Suggested change

Currently, no browsers support this feature.

Currently, no browsers have implemented this feature.

for consistency with previous such notes?

chrisdavidmills · 2025-08-03T15:33:40Z

files/en-us/web/css/frequency/index.md

@@ -48,7 +48,7 @@ Invalid frequency values:

 ## Browser compatibility

-No browser currently supports this feature.
+Currently, no browsers support this feature.


Suggested change

Currently, no browsers support this feature.

Currently, no browsers have implemented this feature.

for consistency with previous such notes?

chrisdavidmills · 2025-08-03T15:33:49Z

files/en-us/web/css/image/image/index.md

@@ -159,7 +159,7 @@ The above will put a semi-transparent black mask over the Firefox logo backgroun

 ## Browser compatibility

-There is no browser implementing this feature.
+Currently, no browsers support this feature.


Suggested change

Currently, no browsers support this feature.

Currently, no browsers have implemented this feature.

for consistency with previous such notes?

chrisdavidmills · 2025-08-03T15:34:05Z

files/en-us/web/css/mask-border-mode/index.md

@@ -44,7 +44,7 @@ mask-border-mode: unset;

 ## Browser compatibility

-This property is not supported by any browser yet.
+Currently, no browsers have implemented this property.


Suggested change

+Currently, no browsers have implemented this property.

Currently, no browsers have implemented this feature.

for consistency with previous such notes?

agreed. going with "feature" in all cases, for consistency and findability

…ross all CSS

estelle · 2025-08-04T07:57:13Z

You are missing:

https://developer.mozilla.org/en-US/docs/Web/CSS/image-resolution

https://developer.mozilla.org/en-US/docs/Web/CSS/line-height-step

I think we should have a uniform template for noting non-support in module landing pages and other pages, and apply it to all other existing notes

line-height-step is in a spec that has not been implemented and that we haven't documented. I added a link to the spec in the see also to help my future self find the spec and document the module. We may want to delete that page (but not in this PR).

Fixed image-resolution. thanks for finding that one.

estelle · 2025-08-04T08:09:59Z

@estelle a few bits for you to consider, but nothing major. Nice work.

I made everything uniform, so touched almost all the content. It may be easier to re-review rather than check previous comments individually.

chrisdavidmills

@estelle Very nice! Just a couple more suggestions for consistency, but approving.

files/en-us/web/css/css_generated_content/index.md

files/en-us/web/css/css_paged_media/index.md

chrisdavidmills · 2025-08-04T08:55:10Z

@estelle a few bits for you to consider, but nothing major. Nice work.

I made everything uniform, so touched almost all the content. It may be easier to re-review rather than check previous comments individually.

Yeah, that's what I did, so all good.

Co-authored-by: Chris Mills <[email protected]>

CSS modules: separate out unsupported features

50d9ad4

estelle requested a review from a team as a code owner July 31, 2025 00:07

estelle requested review from chrisdavidmills and removed request for a team July 31, 2025 00:07

github-actions bot added Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed labels Jul 31, 2025

typo

95f0639

Josh-Cena reviewed Jul 31, 2025

View reviewed changes

chrisdavidmills requested changes Aug 3, 2025

View reviewed changes

estelle and others added 4 commits August 4, 2025 07:38

Merge branch 'main' into 226

aa0f050

remove notes in module landing pages. Make 'not supported' uniform ac…

d206ab4

…ross all CSS

remove no support for stretch and contain

5162acd

final tweaks

a23e235

estelle requested review from chrisdavidmills and Josh-Cena August 4, 2025 07:57

estelle added 2 commits August 4, 2025 09:03

final final tweaks

6838970

final final final tweaks

8b22c16

chrisdavidmills approved these changes Aug 4, 2025

View reviewed changes

files/en-us/web/css/css_generated_content/index.md Outdated Show resolved Hide resolved

files/en-us/web/css/css_paged_media/index.md Outdated Show resolved Hide resolved

estelle and others added 2 commits August 4, 2025 12:42

Update files/en-us/web/css/css_paged_media/index.md

dc9b2b8

Co-authored-by: Chris Mills <[email protected]>

Update files/en-us/web/css/css_generated_content/index.md

550e757

Co-authored-by: Chris Mills <[email protected]>

estelle merged commit bc761c1 into main Aug 4, 2025
12 checks passed

estelle deleted the 226 branch August 4, 2025 12:04

	The specification also defines the {{cssxref("@font-feature-values/font-display", "font-display")}} descriptor which is not yet supported by any browser.
	The specification also defines the {{cssxref("@font-feature-values/font-display", "font-display")}} descriptor, which is not yet supported by any browser.

	> The CSS selectors module introduces the [column combinator](/en-US/docs/Web/CSS/Column_combinator) (`\|\|`) that has not yet been implemented.
	> The CSS selectors module introduces the [column combinator](/en-US/docs/Web/CSS/Column_combinator) (`\|\|`), which is not yet supported by any browser.

	Currently, no browsers support this feature.
	Currently, no browsers have implemented this feature.

	+Currently, no browsers have implemented this property.
	Currently, no browsers have implemented this feature.

CSS modules: separate out unsupported features #40576

CSS modules: separate out unsupported features #40576

Uh oh!

Conversation

estelle commented Jul 31, 2025

Uh oh!

github-actions bot commented Jul 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Josh-Cena left a comment

Choose a reason for hiding this comment

Uh oh!

chrisdavidmills left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

estelle commented Aug 4, 2025

Uh oh!

estelle commented Aug 4, 2025

Uh oh!

chrisdavidmills left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

chrisdavidmills commented Aug 4, 2025

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Jul 31, 2025 •

edited

Loading