WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
247852
Setting CSS shorthands should serialize with the shorthand in cssText
https://bugs.webkit.org/show_bug.cgi?id=247852
Summary
Setting CSS shorthands should serialize with the shorthand in cssText
Tim Nguyen (:ntim)
Reported
2022-11-12 23:23:15 PST
``` const div = document.createElement("div"); div.style.font = "16px Arial"; console.log(div.getAttribute("style")) ``` Actual: ``` font-family: Arial; font-size: 16px; font-style: initial; font-variant-caps: initial; font-weight: initial; font-stretch: initial; line-height: initial; font-size-adjust: initial; font-kerning: initial; font-variant-alternates: initial; font-variant-ligatures: initial; font-variant-numeric: initial; font-variant-east-asian: initial; font-variant-position: initial; font-feature-settings: initial; font-optical-sizing: initial; font-variation-settings: initial; font-palette: initial; ``` Expected: ``` font: 16px Arial; ``` --- System fonts also seem handled incorrectly: ``` font-family: status-bar; font-size: status-bar; font-style: status-bar; font-variant-caps: status-bar; font-weight: status-bar; font-stretch: status-bar; line-height: status-bar; font-size-adjust: initial; font-kerning: initial; font-variant-alternates: initial; font-variant-ligatures: initial; font-variant-numeric: initial; font-variant-east-asian: initial; font-variant-position: initial; font-feature-settings: initial; font-optical-sizing: initial; font-variation-settings: initial; font-palette: initial; ```
Attachments
Add attachment
proposed patch, testcase, etc.
Tim Nguyen (:ntim)
Comment 1
2022-11-12 23:29:49 PST
We have the same problem with `background` and other shorthands fwiw. Oriol, are you aware of any other bugs filed for this?
Oriol Brufau
Comment 2
2022-11-13 01:00:47 PST
I guess the style attribute is just set to div.style.cssText. I recently refactored this, there are various shorthands that are not serialized:
https://searchfox.org/wubkat/rev/8b4aafa95d0aa976fe71b6dd3183242487807bc8/Source/WebCore/css/StyleProperties.cpp#1704-1770
So setting font should set all longhands, it's just that they should be serialized with the shorthand. Not sure if there are existing bugs about this.
Tim Nguyen (:ntim)
Comment 3
2022-11-13 03:10:57 PST
(In reply to Oriol Brufau from
comment #2
)
> I guess the style attribute is just set to div.style.cssText. > I recently refactored this, there are various shorthands that are not > serialized: >
https://searchfox.org/wubkat/rev/8b4aafa95d0aa976fe71b6dd3183242487807bc8/
> Source/WebCore/css/StyleProperties.cpp#1704-1770
Thanks! I think most of the items were unintentional omissions (like I don't see a reason why new props like offset can't serialize with the shorthand).
Radar WebKit Bug Importer
Comment 4
2022-11-19 23:24:13 PST
<
rdar://problem/102555818
>
Karl Dubost
Comment 5
2024-04-07 17:35:31 PDT
***
Bug 271001
has been marked as a duplicate of this bug. ***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug