w3ctag / design-reviews

W3C specs and API reviews
Creative Commons Zero v1.0 Universal
318 stars 55 forks source link

CSS `text-spacing` property and its longhands #907

Closed kojiishi closed 7 months ago

kojiishi commented 9 months ago

Hello TAG!

I'm requesting a TAG review of CSS text-spacing property and their longhands.

This property is a shorthand for setting text-spacing-trim and text-autospace in a single declaration. They control the typographic behavior for Han-derived scripts, specifically Chinese and Japanese. Traditional Han-based Korean is also included but Hangul is usually not included.

Further details:

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), @clqsin45 (implementer), @bfgeek (implementer), @fantasai (spec editor), @frivoal (spec editor)]

torgo commented 8 months ago

Hi @kojiishi we really need an explainer in order to execute a review. Thanks!

kojiishi commented 8 months ago

Updated.

torgo commented 8 months ago

Hi @kojiishi thanks for that – we're reviewing this morning and largely happy with the proposed syntax. Can you also let us know more about the status of this work in the CSS working group and multi-browser support. Are there webkit and Mozilla standards positions on this? Can you also let us know about any W3C internationalization review if this has happened?

kojiishi commented 8 months ago

Can you also let us know more about the status of this work in the CSS working group

@frivoal @fantasai ?

and multi-browser support. Are there webkit and Mozilla standards positions on this?

WebKit said "text-spacing is something we’d like to start implementing soon", and Safari Technology Preview has a runtime flag. @nt1m might be able to provide more.

Mozilla standards-position #903 has a comment "yes" but I'm not sure if this is from a Mozilla's engineer. @jfkthame @masayukinakano ?

Can you also let us know about any W3C internationalization review if this has happened?

@r12a was involved in most of discussions for this property at CSSWG, and W3C Japanese Requirement Task Force (@kidayasuo chairs this TF) as well.

frivoal commented 8 months ago

Can you also let us know more about the status of this work in the CSS working group @frivoal @fantasai ?

The spec as a whole has yet to reach CR, which would indicate that the CSS-WG thinks it's done with it (unless further external feedback comes), so it is premature to claim stability on the whole feature set, and things are quite interconnected overall.

That said, for this particular part, I think we're getting close. I would expect some further tweaking of the details as we continue to iterate (see https://github.com/w3c/csswg-drafts/pull/9503, or https://github.com/w3c/csswg-drafts/issues/9423, or https://github.com/w3c/csswg-drafts/issues/8511 for instance), but I think the way the feature works overall, and its syntax in particular, are unlikely to change much going forward.

ylafon commented 7 months ago

We looked at this during our latest breakout, discussing the "generic" name of text-spacing-trim while it applies only to CJK, in contrast, text-autospace applies to more than the CJK context. Considering also the assessment of stability above, we are happy to close this issue.