Bug 226875

Summary: Web Inspector: CSS variables not handled as case sensitive
Product: WebKit Reporter: Razvan Caliman <rcaliman>
Component: Web InspectorAssignee: Nikita Vasilyev <nvasilyev>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, hi, inspector-bugzilla-changes, joepeck, nvasilyev, pangle, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Web Inspector with different case CSS variables
none
[HTML] Reduction
none
Patch v1.0 none

Razvan Caliman
Reported 2021-06-10 06:01:00 PDT
Created attachment 431066 [details] Web Inspector with different case CSS variables From https://www.w3.org/TR/css-variables-1/ Unlike other CSS properties, custom property names are case-sensitive. EXAMPLE 2: While both --foo and --FOO are valid, they are distinct properties - using var(--foo) will refer to the first one, while using var(--FOO) will refer to the second. --- Web Inspector lowercases all CSS custom property names and this influences behavior. For example: ``` div { --color: green; --COLOR: red; color: var(--color); } ``` Variables are mistakenly marked as overwritten in Styles pane. Variables with different case are not shown in Computed pane > Variables section. (Resolving variable names to their value is handled correctly, though).
Attachments
Web Inspector with different case CSS variables (223.46 KB, image/png)
2021-06-10 06:01 PDT, Razvan Caliman
no flags
[HTML] Reduction (339 bytes, text/html)
2021-06-14 10:28 PDT, Nikita Vasilyev
no flags
Patch v1.0 (4.92 KB, patch)
2021-06-14 11:14 PDT, Patrick Angle
no flags
Nikita Vasilyev
Comment 1 2021-06-14 10:28:20 PDT
Created attachment 431343 [details] [HTML] Reduction
Patrick Angle
Comment 2 2021-06-14 11:14:34 PDT
Created attachment 431346 [details] Patch v1.0
Devin Rousso
Comment 3 2021-06-14 11:56:02 PDT
Comment on attachment 431346 [details] Patch v1.0 r=me
EWS
Comment 4 2021-06-14 13:01:37 PDT
Committed r278848 (238794@main): <https://commits.webkit.org/238794@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 431346 [details].
Radar WebKit Bug Importer
Comment 5 2021-06-14 13:02:19 PDT
Note You need to log in before you can comment on or make changes to this bug.