Skip to content

feat: updated the value for text-decoration-inset - #1087

Open
dletorey wants to merge 2 commits into
mdn:mainfrom
dletorey:44840-text-decoration-inset-percentages
Open

feat: updated the value for text-decoration-inset#1087
dletorey wants to merge 2 commits into
mdn:mainfrom
dletorey:44840-text-decoration-inset-percentages

Conversation

@dletorey

@dletorey dletorey commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Description

  • updated the value for text-decoration-inset from <length> to <length-percentage>

Motivation

Related issues and pull requests

@dletorey
dletorey requested review from a team as code owners August 5, 2026 13:27
@dletorey
dletorey requested a review from dipikabh August 5, 2026 13:27
@dletorey dletorey changed the title updated the value for text-decoration-inset feat: updated the value for text-decoration-inset Aug 5, 2026
Comment thread css/properties.json Outdated

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.

Oddly, this entry in the spec still says "N/A":

Percentages: N/A

whereas it could possibly be updated to something like "referToElementFontSize".

Same for "computed". It might need to be updated to "asSpecified" but spec still says:

Computed value: specified keyword or absolute length

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This has now changed and moved on since. referToSizeOfElement seems to be the correct value now

@dipikabh dipikabh 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.

Your update looks good. Thank you!

@dletorey
dletorey requested a review from dipikabh August 10, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants