Implement DevTools support for Properties & Values API

NEW
Unassigned

Status

enhancement
P3
normal
3 years ago
10 months ago

People

(Reporter: jyc, Unassigned)

Tracking

(Depends on 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Implement support for the CSS Properties & Values API ( https://drafts.css-houdini.org/css-properties-values-api/ ) in devtools.

Placeholder bug, still working out what this would entail!
Summary: Implement support for Properties & Values API → Implement DevTools support for Properties & Values API
Component: Developer Tools → Developer Tools: Inspector
Thanks for the help!
Tentative list of things I've seen:

- The Computed tab in the Inspector currently only seems to show standard properties. Because custom properties can be animated and that requires computing them, would it make sense to include them too? The patches add a new computed variables style struct specifically for registered custom properties, so this should be not hard.
- The Rules tab strikes out declarations that don't type correctly. I think that this needs to be updated for custom properties. If there is a cycle in variable declarations, that becomes invalid at compute time -- should these declarations get strikes too? The patches modify CSSVariableDeclarations so it's possible to get which variable declaration is actually used, so I think that this should be not hard either. If a declaration involves a cycle, then it will be used but will have no computed value.
Triaging as enhancement for now.
In terms of priority, could you help me assess this? What's the priority of the corresponding platform bug to implement custom properties (bug 1273706)?
When do you expect it to land? Will it be enabled by default or ship hidden behind a pref?
Severity: normal → enhancement
Flags: needinfo?(jchan)
I'll steal the needinfo on this.

(In reply to Patrick Brosset <:pbro> (PTO until August 16, not doing reviews) from comment #2)
> Triaging as enhancement for now.

That sounds good.

> In terms of priority, could you help me assess this? What's the priority of
> the corresponding platform bug to implement custom properties (bug 1273706)?

Probably low-priority from a devtools perspective. (Zero priority until the dependent bug has landed and there's something to test.)  Priority level will be in part based on how well these custom properties get already exposed in existing devtools.

> When do you expect it to land?

Within the next few weeks, probably. (undergoing review now.)

> Will it be enabled by default or ship hidden
> behind a pref?

It'll be hidden behind a pref (tentatively named "layout.css.properties_and_values.enabled") for now.

I expect we'll do the standard "enable on Nightly/Aurora for testing, but not in release builds" partial-enabling after it's gotten some baking & opt-in nightly testing.
Flags: needinfo?(jchan)
Priority: -- → P3
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.