Re: [csswg-drafts] [css-text-decor] text-underline-position auto in vertical text (#1198)

The CSS Working Group just discussed `[css-text-decor] text-underline-position auto in vertical text`, and agreed to the following:

* `RESOLVED: add auto value for text-emphasis-position, and change the meaning of text-underline-position: auto to care about left vs right in vertical text`

<details><summary>The full IRC log of that discussion</summary>
&lt;chrishtr> fantasai: the initial value of text-underline-position is auto, which is defined as "find a good place to put the underline". Three options there: (1) under alphabetical baseline, (2) fully below text (good for lots-of-descenders cases), (3) for vertical text on the RHS<br>
&lt;chrishtr> fantasai: auto value is defined in the spec about 'how far down below the text', but doesn't say things about flippinng<br>
&lt;chrishtr> fantasai: the current spec says "at or below"<br>
&lt;chrishtr> fantasai: in order to handle language-specific aspects, there is a default UA style sheet that for Chinese and Japanese and Korean there are differences for those languages<br>
&lt;fantasai> https://drafts.csswg.org/css-text-decor-3/#default-stylesheet<br>
&lt;chrishtr> fantasai: a couple of implementations do this<br>
&lt;chrishtr> fantasai: should we change the spec to mention these things?<br>
&lt;chrishtr> fantasai: or should we stick with the UA stylesheet approach?<br>
&lt;fantasai> https://www.w3.org/TR/css-text-decor-4/#text-emphasis-position-property<br>
&lt;chrishtr> fantasai: propose that we keep the spec as-is<br>
&lt;fantasai> https://www.w3.org/TR/css-text-decor-4/#text-underline-position-property<br>
&lt;Rossen3> q?<br>
&lt;chrishtr> fantasai: this would require some implementations to change though<br>
&lt;chrishtr> chrishtr: which implementations would need to change?<br>
&lt;chrishtr> fantasai: chrome and firefox are language-sensitive for auto, and webkit uses the default UA style sheet<br>
&lt;chrishtr> rossen: does this mean that webkit needs to change?<br>
&lt;chrishtr> florian: other way around, it would mean chrome and firefox need to change?<br>
&lt;chrishtr> florian: since the two approaches both exist it seems going either way would be web compatible<br>
&lt;emilio> q+<br>
&lt;chrishtr> rossen: sounds like a low-ROI change<br>
&lt;chrishtr> rossen: is it a problem in practice?<br>
&lt;Rossen3> ack emilio<br>
&lt;chrishtr> emilio: I think we should try to go for the firefox/chrome approach<br>
&lt;chrishtr> emilio: avoids weird styles change in ways that developers might not expect<br>
&lt;chrishtr> emilio: we had the same problem with quotes if I'm remembering correctly<br>
&lt;chrishtr> fantasai: that was the first time we had a language-aware value<br>
&lt;chrishtr> emilio: reusing that mechanism for this makes sense, but don't have a strong opinion<br>
&lt;chrishtr> fantasai: if there is a strong need for these things they we could introduce auto keywords for other things, otherwise UA stylesheet for this case?<br>
&lt;chrishtr> jfkthame: text decoration skip ink does something also, seems to me auto is the cleanest approach<br>
&lt;fantasai> s/other things/text-emphasis-position/<br>
&lt;florian> s/and firefox need to change?/and firefox need to change if we keep the spec unchanged?<br>
&lt;chrishtr> ntim: aligning with text-emphasis-position makes sense to me, and it doesn't have an auto value. i.e. that feature uses UA style sheet rules<br>
&lt;chrishtr> chrishtr: is that true for all browsers?<br>
&lt;chrishtr> fantasai: yes, because there is no auto keyword<br>
&lt;dholbert> s/does something/does something language-specific/ (in jfkthame's minutes above)<br>
&lt;chrishtr> jfkthame: it would make sense to me to add auto to that property also<br>
&lt;chrishtr> florian: that would be a change in all browsers<br>
&lt;chrishtr> jfkthame: yes but that could be an improvement<br>
&lt;chrishtr> ntim: is it a common use case to use the auto value to override a non-default value?<br>
&lt;chrishtr> ntim: if not, then the UA style sheet does the job just fine<br>
&lt;chrishtr> florian: we can achieve the effect we want with the UA style sheet, or with auto. both approaches yield the desired result from an author point of view<br>
&lt;chrishtr> florian: from an author point of view, both work. Agree that it's odd for two very similar properties to have different approaches, agree it would be best to be consistent.<br>
&lt;fantasai> A) Keep spec as-is, update Gecko + Blink to match (using UA stylesheet for language switch)<br>
&lt;chrishtr> fantasai: option a: keep spec as-is, update gecko &amp; chromium to match. option b: change spec, change webkit to match.<br>
&lt;fantasai> B) Introduce to text-emphasis-position and use it in both text-emphasis-position and text-underline-position to effect language switches<br>
&lt;ntim> Option b requires changing text-emphasis-position in all browsers too<br>
&lt;fantasai> s/Introduce/Introduce auto/<br>
&lt;fantasai> C) Adopt inconsistent behavior: text-underline-position uses 'auto' and text-emphasis-position uses UA stylesheet<br>
&lt;TabAtkins> abstain, no opinion<br>
&lt;emilio> B<br>
&lt;fantasai> POLL: A, B, or C?<br>
&lt;vmpstr> abstain<br>
&lt;chrishtr> B<br>
&lt;jfkthame> B, A, C<br>
&lt;astearns> abstain<br>
&lt;ntim> A, B, C<br>
&lt;fantasai> A, B, C<br>
&lt;ydaniv> abstain<br>
&lt;miriam> abstain<br>
&lt;florian> indifferent between A and B, dislike of C<br>
&lt;dholbert> B, A, C<br>
&lt;schenney> B, A, C<br>
&lt;dbaron> neutral on A vs B, prefer them to C<br>
&lt;oriol> abstain<br>
&lt;rachelandrew> abstain, no strong opinion<br>
&lt;DavidA> abstain<br>
&lt;kizu> abstain<br>
&lt;kbabbitt> abstain<br>
&lt;flackr> abstain<br>
&lt;chrishtr> proposed resolution: add auto value for text-emphasis-position, and change the meaning of text-underline-position: auto to care about left vs right in vertical text<br>
&lt;florian> wfm<br>
&lt;chrishtr> fantasai: one side effect of the proposed resolution is that the computed style is less transparent to the developer, vs inspecting the UA style sheet<br>
&lt;emilio> q+<br>
&lt;flackr> q+<br>
&lt;chrishtr> emilio: you have the opposite argument with making initial do the right thing, right?<br>
&lt;chrishtr> emilio: there are arguments in both directions in this dimension<br>
&lt;Rossen3> ack emilio<br>
&lt;chrishtr> emilio: being able to set something reasonable via resets in the style sheet, I mean<br>
&lt;chrishtr> emilio: would expect the initial value to do the right thing - resetting gets rid of UA style sshets<br>
&lt;chrishtr> s/sshets/sheets/<br>
&lt;chrishtr> jftkhame: does seem an auto keyword should do the right thing<br>
&lt;Rossen3> ack flackr<br>
&lt;chrishtr> flackr: what would a UA style sheet rule setting this look like?<br>
&lt;fantasai> https://www.w3.org/TR/css-text-decor-4/#default-stylesheet<br>
&lt;chrishtr> fantasai: current default style sheet rules ^^<br>
&lt;florian> :root:lang(zh), [lang|=zh] { text-emphasis-position: under right; }<br>
&lt;florian> [lang|=ja], [lang|=ko]     { text-emphasis-position: over right; }<br>
&lt;chrishtr> flackr: writing direction doesn't affect this?<br>
&lt;chrishtr> fantasai: there are two keywords to set the position<br>
&lt;chrishtr> flackr: Thanks. I'm still in favor of option B<br>
&lt;ntim> I'm not objecting, but I can't give a guarantee we can implement option A anytime soon<br>
&lt;chrishtr> RESOLVED: add auto value for text-emphasis-position, and change the meaning of text-underline-position: auto to care about left vs right in vertical text<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/1198#issuecomment-2358986319 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 18 September 2024 17:01:12 UTC