[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