262762 2023-10-05 22:22:20 -0700 Implement percentage values for letter-spacing and word-spacing [css-text-4] 2024-03-01 06:46:07 -0800 1 1 1 Unclassified WebKit Text Safari 17 Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=270347 https://github.com/web-platform-tests/wpt/pull/44882 InRadar P2 Normal --- 1 fantasai.bugs webkit-unassigned karlcow mmaxfield ntim webkit-bug-importer oldest_to_newest 1983107 0 fantasai.bugs 2023-10-05 22:22:20 -0700 Background: CSS Text Level 4 <https://www.w3.org/TR/css-text-4/> adds support for percentage values of word-spacing and letter-spacing. These inherit as percentages, so that they are always relative to the currently-effective font-size. Current Status: WebKit has support for percentage values of word-spacing (but not calc() values) but calculates them against the width of a space instead of the font-size, and doesn't inherit them properly; and has no support for percentage letter-spacing. Proposal: Re-implement percentage values of word-spacing per spec, add support for percentage values of letter-spacing, and implement support for calc(). Compat Risks: Firefox also implements percentage word-spacing in a similar way to WebKit, but Chrome doesn't implement percentage word-spacing, so it's unlikely that Web content depends on it. There could be some lock-in in single-engine environments, but because the current implementation doesn't implement inheritance properly, the benefit of using % over fixed values is not fully realized in the current implementation, so the incentive to use them over 'em' values (which have existed since CSS1) is limited. 1983109 1 fantasai.bugs 2023-10-05 22:27:09 -0700 rdar://114538918 1983111 2 webkit-bug-importer 2023-10-05 22:27:54 -0700 <rdar://problem/116562262> 1983480 3 ap 2023-10-07 13:46:26 -0700 rdar://114538918 1986783 4 ntim 2023-10-22 23:58:49 -0700 https://github.com/WebKit/WebKit/pull/18771/ 1995996 5 ews-feeder 2023-11-30 13:52:32 -0800 Committed 271357@main (6abbfb00701a): <https://commits.webkit.org/271357@main> Reviewed commits have been landed. Closing PR #18771 and removing active labels.