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

The CSS word-break: auto-phrase property #891

Closed
1 task done
kojiishi opened this issue Sep 1, 2023 · 6 comments
Closed
1 task done

The CSS word-break: auto-phrase property #891

kojiishi opened this issue Sep 1, 2023 · 6 comments
Assignees
Labels
Resolution: satisfied The TAG is satisfied with this design Topic: CSS Venue: CSS WG

Comments

@kojiishi
Copy link

kojiishi commented Sep 1, 2023

Hello TAG!

I'm requesting a TAG review of the CSS word-break: auto-phrase property..

This feature enables keeping natural phrases (of multiple words) together when line breaking.

Further details:

  • I have reviewed the TAG's Web Platform Design Principles
  • Relevant time constraints or deadlines: Implementation complete.
  • The group where the work on this specification is currently being done: CSSWG
  • The group where standardization of this work is intended to be done (if current group is a community group or other incubation venue): CSSWG
  • Major unresolved issues with or opposition to this specification: None
  • This work is being funded by: Blink implementation is being done by Google.

We'd prefer the TAG provide feedback as (please delete all but the desired option):
💬 leave review feedback as a comment in this issue and @-notify [@kojiishi (implementer), @bfgeek (implementer), @fantasai (spec editor), @frivoal (spec editor)]

@rhiaro
Copy link
Contributor

rhiaro commented Oct 9, 2023

Hi @kojiishi please could you update your request with the rest of the template filled in?

@rhiaro rhiaro self-assigned this Oct 9, 2023
@kojiishi
Copy link
Author

Hi @rhiaro, sorry I missed some fields to fill in. Done.

@torgo
Copy link
Member

torgo commented Oct 11, 2023

Hi @kojiishi we really need an explainer in order to do a review. Also can you please clarify at what stage this is in in the CSS working group? Is this still under discussion or has this reached consensus?

@kojiishi
Copy link
Author

kojiishi commented Oct 11, 2023

Updated.

@frivoal
Copy link

frivoal commented Oct 12, 2023

Can you please clarify at what stage this is in in the CSS working group?

Quite settled. We might have to address follow up issues if raised, but the core of the discussion has settled, and the spec is where we expect it to be for this feature.

Explainer

In addition to the document shared by Koji, appendix H in the spec, as well as example 22 do give some examples and some high level description of what this is.

@rhiaro
Copy link
Contributor

rhiaro commented Nov 22, 2023

Thanks @kojiishi. We see no architectural concerns, and are happy to close this review.

We would like to note that you haven't filled out the privacy and security questionnaire, but have linked to the security and privacy sections in your spec, the first of which contains nothing, and the second of which contains one sentence:

This specification leaks the user’s installed hyphenation and line-breaking dictionaries.

We were wondering if you could expand on the risks here, and if there are mitigations for this that could/should be elaborated on in this section, or if it's not really a concern, then why not?

@rhiaro rhiaro added Resolution: satisfied The TAG is satisfied with this design Progress: propose closing we think it should be closed but are waiting on some feedback or consensus labels Nov 22, 2023
@rhiaro rhiaro closed this as completed Dec 20, 2023
@plinss plinss removed the Progress: propose closing we think it should be closed but are waiting on some feedback or consensus label Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: satisfied The TAG is satisfied with this design Topic: CSS Venue: CSS WG
Projects
None yet
Development

No branches or pull requests

7 participants