1271191 - Support CSS #rgba and #rrggbbaa color syntax in the inspector

Status: RESOLVED FIXED

(DevTools :: Inspector, enhancement)

Description

[Patrick Brosset <:pbro>]

Bug 567283 brings #rgba and #rrggbbaa color syntax to CSS.
See the intent to ship thread: https://groups.google.com/forum/#!topic/mozilla.dev.platform/00Tq2s58GwA

We need to make sure devtools recognizes these new color values as colors (shows color swatches in the rule-view and computed-view), and is able to convert between these new formats and other formats (for instance, shift-clicking on a color swatch should cycle through the different formats).

In particular, https://dxr.mozilla.org/mozilla-central/source/devtools/client/shared/css-color.js is impacted I think.

Comment 1

[Tom Tromey :tromey]

Also a spot in inplace-editor, see _incHexColor

Comment 2

[Julian Descottes [:jdescottes]]

Inspector bug triage. Filter on CLIMBING SHOES.

Comment 3

[Patrick Brosset <:pbro>]

The intent to ship email was sent rather late, and the new CSS syntax actually already landed in mozilla-central, and will be shipping with 49. It would be nice to have devtools ready at the same time.

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

Attachment: MozReview Request: Bug 1271191 - Add support CSS #rgba and #rrggbbaa color syntax in the inspector. r=miker

Add support for both syntax in CssColor
Add support in value increment
Edit several tests to make sure we handle these syntaxes all across the inspector (display swatch,
color increment, unit switching, …).

Review commit: https://reviewboard.mozilla.org/r/52117/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/52117/

Comment 5

[Nicolas Chevobbe [:nchevobbe]]

I pushed my patch to TRY (only one platform) to ensure it doesn't break something (and it seems good) : https://treeherder.mozilla.org/#/jobs?repo=try&revision=da19cc727137

Comment 6

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 8751587 [details]
MozReview Request: Bug 1271191 - Add support CSS #rgba and #rrggbbaa color syntax in the inspector. r=miker

https://reviewboard.mozilla.org/r/52117/#review49968

Nice and even completely green on try... an easy r+.

Comment 7

[Nicolas Chevobbe [:nchevobbe]]

TRY is over (https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e8df9b706bc&selectedJob=20956794). There's  lots of failure, but I checked and I don't see how any of them are related to my patch.

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/8751a7511341

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/8751a7511341