[webkit-changes] [WebKit/WebKit] ea1eac: text-emphasis: Do not paint emphasis marks on unic...

Ahmad Saleem noreply at github.com
Mon Jan 6 11:03:41 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ea1eac388c7648b2a257c99e10a3cf0504e1ec99
      https://github.com/WebKit/WebKit/commit/ea1eac388c7648b2a257c99e10a3cf0504e1ec99
  Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
  Date:   2025-01-06 (Mon, 06 Jan 2025)

  Changed paths:
    M LayoutTests/TestExpectations
    M Source/WTF/wtf/unicode/CharacterNames.h
    M Source/WebCore/platform/graphics/FontCascade.cpp

  Log Message:
  -----------
  text-emphasis: Do not paint emphasis marks on unicode punctuations

https://bugs.webkit.org/show_bug.cgi?id=285405
rdar://142387538

Reviewed by Darin Adler.

This patch aligns WebKit with Gecko / Firefox and Blink / Chromium and Web Specification [1]:

[1] https://drafts.csswg.org/css-text-decor-3/#text-emphasis-style-property

"Punctuation--specifically, any characters that belong to the Unicode P* general category..."

and was clarified in: https://github.com/w3c/csswg-drafts/issues/839

Inspired by: https://source.chromium.org/chromium/chromium/src/+/dba2b53e93f71fa2df02d72a42a42371a63c695b and
https://phabricator.services.mozilla.com/D219099

This patch adds unicode punctuations list, which should not receive `text emphasis`
painting / styling as well those which should (matching Firefox implementation).

Despite aligning WebKit with Firefox implementation, we are not matching expected output
to enable `text-emphasis-punctuation-3.html` but we are not regressing behavior as well wit
this change.

* LayoutTests/TestExpectations:
* Source/WebCore/platform/graphics/FontCascade.cpp:
(WebCore::FontCascade::canReceiveTextEmphasis):
* Source/WTF/wtf/Unicode/CharacterNames.h: Sorted alphabetically as well.

Canonical link: https://commits.webkit.org/288476@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list