Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed what the property overflow-wrap applies to #31727

Merged
merged 3 commits into from
Jan 16, 2024
Merged

Fixed what the property overflow-wrap applies to #31727

merged 3 commits into from
Jan 16, 2024

Conversation

mcmimik
Copy link
Contributor

@mcmimik mcmimik commented Jan 15, 2024

Description

According to the current standard, the property is not applied to inline elements, but to text. See: https://www.w3.org/TR/css-text-3/#overflow-wrap-property

Motivation

Aligning MDN documentation with current version of W3C standard.

Additional details

In the 2018 version of the "CSS Text Module Level 3", the overflow-wrap property was applied to inline boxes. This was updated in the 2020 revision, changing its application to text.

Related issues and pull requests

mdn/data#705

According to the current standard, the property is not applied to inline elements, but to text. See: https://www.w3.org/TR/css-text-3/#overflow-wrap-property
@mcmimik mcmimik requested a review from a team as a code owner January 15, 2024 13:44
@mcmimik mcmimik requested review from estelle and removed request for a team January 15, 2024 13:44
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Jan 15, 2024
Copy link
Contributor

github-actions bot commented Jan 15, 2024

Preview URLs

(comment last updated: 2024-01-16 16:44:41)

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

Good catch. Thank you.
Also, congratulations on your first merged MDN content PR.
Welcome to the team!

@estelle estelle merged commit 30a4f02 into mdn:main Jan 16, 2024
8 checks passed
dipikabh pushed a commit to dipikabh/content that referenced this pull request Jan 17, 2024
Clarified what the property applies to

According to the current standard, the property is not applied to inline elements, but to text. See: https://www.w3.org/TR/css-text-3/#overflow-wrap-property
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants