[csswg-drafts] [css-overflow][css-text-decor] should text-decoration include ellipsis? (#6531)

delan has just created a new issue for https://github.com/w3c/csswg-drafts:

== [css-overflow][css-text-decor] should text-decoration include ellipsis? ==
Given the following ([demo](https://bucket.daz.cat/work/igalia/0/18.html)), should the decorations extend to cover the “…” or “,,,”?

main {
    text-decoration: underline;
main > p {
    text-decoration: overline;
    overflow: hidden;
    white-space: nowrap;
    width: 3em;
    <p style='text-overflow: ellipsis;'>hello world</p>
    <p style='text-overflow: ",,,";'>hello world</p>

No known impl says yes, but I couldn’t find it specified anywhere in css-overflow or css-text-decor. @mrego thought it would be good to ask after seeing [this code in Chromium](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/paint/ng/ng_text_painter.cc;l=120-125;drc=a64cf4accde5e5412f228a7b40c6fd3c9f0c9538):

  if (style.TextDecorationsInEffect() == TextDecoration::kNone ||
      // Ellipsis should not have text decorations. This is not defined, but
      // 4 impls do this.
      text_item.IsEllipsis()) {
    return absl::nullopt;

Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6531 using your GitHub account

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

Received on Friday, 20 August 2021 12:13:12 UTC