WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
224010
Web Inspector: Autocomplete experimental CSS Color values (hwb, lch, lab, color-mix, color-contrast)
https://bugs.webkit.org/show_bug.cgi?id=224010
Summary
Web Inspector: Autocomplete experimental CSS Color values (hwb, lch, lab, col...
Nikita Vasilyev
Reported
2021-03-31 12:27:01 PDT
Web Inspector should show completions for these when the corresponding experimental settings are enabled.
Attachments
Patch
(3.66 KB, patch)
2021-03-31 12:30 PDT
,
Nikita Vasilyev
no flags
Details
Formatted Diff
Diff
Patch
(1.76 KB, patch)
2021-03-31 12:55 PDT
,
Nikita Vasilyev
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2021-03-31 12:27:11 PDT
<
rdar://problem/76065217
>
Nikita Vasilyev
Comment 2
2021-03-31 12:30:35 PDT
Created
attachment 424801
[details]
Patch
Joseph Pecoraro
Comment 3
2021-03-31 12:44:37 PDT
Comment on
attachment 424801
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=424801&action=review
> Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js:166 > +WI.CSSKeywordCompletions.getExperimentalProperties = function(properties) {
Style: Put the `{` on the next line?
> Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js:169 > + if (CSS.supports(test))
The `CSS.supports` is neat, but why not do it all time? You could be on a macOS system that doesn't support the syntax and inspecting an iOS target that does support it. In that case we'd probably still want completion to work. So might as well do it all the time.
Nikita Vasilyev
Comment 4
2021-03-31 12:55:17 PDT
Created
attachment 424808
[details]
Patch
Blaze Burg
Comment 5
2021-03-31 15:10:00 PDT
Comment on
attachment 424808
[details]
Patch r=me
EWS
Comment 6
2021-03-31 15:31:58 PDT
Committed
r275310
: <
https://commits.webkit.org/r275310
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 424808
[details]
.
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