Skip to content

Reset-only sub-properties - #45683

Merged
estelle merged 4 commits into
mdn:mainfrom
estelle:resetprops
Sep 17, 2026
Merged

estelle merged 4 commits into
mdn:mainfrom
estelle:resetprops

Conversation

@estelle

@estelle estelle commented Sep 13, 2026

Copy link
Copy Markdown
Member

Several shorthand properties, like animation accept a limited number of properties while being reset-only for other properties. We should include a section after ### Constituent properties that lists the reset-only properties that are reset by the shorthand.

fixes #45591

@estelle
estelle requested a review from a team as a code owner September 13, 2026 18:51
@estelle
estelle requested review from dipikabh and removed request for a team September 13, 2026 18:51
@github-actions github-actions Bot added Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed labels Sep 13, 2026
@github-actions

github-actions Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (5 pages)

Flaws (2)

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

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/CSS/Reference/Properties/animation
Title: animation CSS property
Flaw count: 2

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/animation-delay-end which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/animation-trigger which doesn't resolve

(comment last updated: 2026-09-17 10:46:29)

@github-actions github-actions Bot added the merge conflicts 🚧 [PR only] label Sep 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions Bot removed the merge conflicts 🚧 [PR only] label Sep 14, 2026

@chrisdavidmills chrisdavidmills left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. One small structural snafu to fix, but looking good apart from that.

Have you added this section to the reference template(s)?

- {{cssxref("background-repeat")}}
- {{cssxref("background-size")}}

## Syntax

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have lost the "Syntax" heading.

Comment thread files/en-us/web/css/reference/properties/background/index.md
@estelle

estelle commented Sep 17, 2026

Copy link
Copy Markdown
Member Author

Have you added this section to the reference template(s)?

Done in #45759. Thanks for the reminder.

@estelle
estelle merged commit 3f221b9 into mdn:main Sep 17, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reset-only constituent properties (Shorthand reset-only properties?)

3 participants