Summary: | Entirely remove support for -apple-trailing-word | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Nguyen (:ntim) <ntim> | ||||||
Component: | CSS | Assignee: | Tim Nguyen (:ntim) <ntim> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | annulen, benjamin, cdumez, cmarcelo, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, jbedard, joepeck, macpherson, menard, mmaxfield, ryuan.choi, sergio, simon.fraser, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=196575 https://bugs.webkit.org/show_bug.cgi?id=235961 |
||||||||
Bug Depends on: | 196525 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Tim Nguyen (:ntim)
2021-12-24 14:50:24 PST
Created attachment 450431 [details]
Patch
I'm of 2 minds about this. On one hand I want to kill -apple-trailing-word with fire. On the other hand, it looks like it's still used by at least a few fairly high-visibility repos. cssinjs and reacttips-dev seem worth at least filing issues on. Comment on attachment 450431 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=450431&action=review > Source/WebCore/css/CSSProperties.json:-7650 > - "-apple-trailing-word": { Maybe leave it in but mark as "Removed". Created attachment 450505 [details]
Patch
Committed r288869 (246621@main): <https://commits.webkit.org/246621@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 450505 [details]. Looks like this is causing webkitpy failures: https://results.webkit.org/?suite=webkitpy-tests&test=webkitpy.w3c.test_converter_unittest.W3CTestConverterTest.test_convert_prefixed_properties Not really sure how that happens.... |