Bug 229086 - Unprefix -webkit-text-size-adjust
Summary: Unprefix -webkit-text-size-adjust
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: https://drafts.csswg.org/css-size-adj...
Keywords: InRadar
Depends on:
Blocks: 217522
  Show dependency tree / graph
 
Reported: 2021-08-13 11:07 PDT by Sam Sneddon [:gsnedders]
Modified: 2024-04-05 05:44 PDT (History)
21 users (show)

See Also:

Attachments
Patch (50.15 KB, patch)
2021-11-30 21:23 PST, Matt Woodrow 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Sneddon [:gsnedders] 2021-08-13 11:07:24 PDT 
I think this already matches the standard property, so I don't think there's any reason not to just unprefix it (with an alias)?
Comment 1 Radar WebKit Bug Importer 2021-08-20 11:08:19 PDT 
<rdar://problem/82177514>
Comment 2 Matt Woodrow 2021-11-30 21:23:35 PST 
Created attachment 445517 [details]
Patch
Comment 3 Simon Fraser (smfr) 2021-11-30 21:28:49 PST 
Is there a stable enough spec whose behavior we match?
Comment 4 Simon Fraser (smfr) 2021-11-30 21:29:49 PST 
Comment on attachment 445517 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=445517&action=review

> Source/WebCore/css/CSSComputedStyleDeclaration.cpp:3410
> +            fprintf(stderr, "Looking up text size adjust\n");

Ooops.
Comment 5 Myles C. Maxfield 2021-11-30 22:08:11 PST 
(In reply to Simon Fraser (smfr) from comment #3)
> Is there a stable enough spec whose behavior we match?

Not really.
Comment 6 Tim Nguyen (:ntim) 2021-12-24 10:39:24 PST 
I think this is mostly missing the <percentage> syntax to match the standard: https://drafts.csswg.org/css-size-adjust/#adjustment-control
Comment 7 Tim Nguyen (:ntim) 2021-12-24 10:43:48 PST 
Oh wait, actually no, we do support the percentage syntax.
Comment 8 Tim Nguyen (:ntim) 2021-12-24 10:45:15 PST 
We should definitely attempt to make the property animatable as one of the WPT show (and the spec as well).