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

[css-text] All IN characters? #2447

Closed
r12a opened this issue Mar 16, 2018 · 3 comments
Closed

[css-text] All IN characters? #2447

r12a opened this issue Mar 16, 2018 · 3 comments
Labels
Closed Accepted as Editorial Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-text-3 Current Work i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response. Tested Memory aid - issue has WPT tests Tracked in DoC

Comments

@r12a
Copy link
Contributor

r12a commented Mar 16, 2018

https://drafts.csswg.org/css-text-3/#line-break-property

• breaks between inseparable characters such as ‥ U+2025, … U+2026 i.e.
characters with the Unicode Line Break property IN. (See LineBreak.txt
in [UNICODE].)

The quoted text above is slightly ambiguous. I took it to mean that this rule "applies to all characters with the Unicode Line Break property IN, such as ‥ U+2025, and … U+2026"

The other possible meaning, given the order in the spec, is that this rule applies to characters such as ‥ U+2025, … U+2026 and those two are characters with the Unicode Line Break property IN

I suggest (if i understood correctly) rewording to say:

• breaks between characters with the Unicode Line Break property IN, i.e. inseparable characters such as ‥ U+2025, and … U+2026. (See LineBreak.txt in [UNICODE].)

@r12a r12a added the i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response. label Mar 16, 2018
@xfq xfq added the css-text-3 Current Work label May 6, 2020
fantasai added a commit that referenced this issue Jul 2, 2020
@fantasai
Copy link
Collaborator

fantasai commented Jul 2, 2020

Added parentheses to disambiguate the sentence parsing. :) Lmk if this is acceptable~

@r12a
Copy link
Contributor Author

r12a commented Jul 2, 2020

Ok, thanks. You can close this. (Should i close it, or do you prefer to do it?)

@frivoal frivoal added Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. and removed Commenter Response Pending labels Jul 2, 2020
@frivoal
Copy link
Collaborator

frivoal commented Jul 2, 2020

(Should i close it, or do you prefer to do it?)

Either way works, but it's done now :)

@frivoal frivoal closed this as completed Jul 2, 2020
@frivoal frivoal added Tested Memory aid - issue has WPT tests and removed Needs Testcase (WPT) labels Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted as Editorial Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-text-3 Current Work i18n-tracker Group bringing to attention of Internationalization, or tracked by i18n but not needing response. Tested Memory aid - issue has WPT tests Tracked in DoC
Projects
None yet
Development

No branches or pull requests

4 participants