Skip to content

[css-text-4] Replace <length-percentages> by <length-percentage> #7301

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

Closed
cdoublev opened this issue May 23, 2022 · 4 comments
Closed

[css-text-4] Replace <length-percentages> by <length-percentage> #7301

cdoublev opened this issue May 23, 2022 · 4 comments
Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-text-4

Comments

@cdoublev
Copy link
Collaborator

The word-spacing property is defined to accept <length-percentages> as a value. I think it should be <length-percentage>, as defined for the value field in its definition table.

Value: normal | <length-percentage>

<length-percentages> Specifies extra spacing in addition to the intrinsic inter-word spacing defined by the font.

@SaarthakMaini
Copy link
Contributor

I think the corresponding issue has already been resolved

@cdoublev
Copy link
Collaborator Author

<dt><dfn>&lt;length-percentages&gt;</dfn>

@SaarthakMaini
Copy link
Contributor

Hello,
I created a pull request regarding the same but I am getting error checks.
My contribution is non-substantive, so how can my contribution get accepted?
Thank you!

@fantasai
Copy link
Collaborator

fantasai commented Dec 29, 2022

@SaarthakMaini With admin privileges? I'm not sure how to do it properly, but anyway it's merged.

Thanks @cdoublev for reporting, and @SaarthakMaini fixing, the error. ^_^

@fantasai fantasai added Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. and removed Needs Edits labels Dec 29, 2022
jakearchibald pushed a commit to jakearchibald/csswg-drafts that referenced this issue Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-text-4
Projects
None yet
Development

No branches or pull requests

4 participants