1132859 - [computed view] Logical CSS properties are not handled

Status: NEW

(DevTools :: Inspector, defect, P3)

Description

[Sebastian Zartner [:sebo]]

Multiple logical CSS properties as of the CSS Logical Properties spec[1] got introduced lately (check the 'See Also' field).

These properties are mapped to physical properties, which is currently not recognized by the Computed side panel.

The panel should list the physical properties (without having 'Browser styles' checked) and allow to expand them to display the logical properties.

E.g. for the given test case I'd expect to see 'margin-top   50px' and when it's expanded it should list 'margin-block-start > 50px' resp. 'padding-bottom   20px' and when it's expanded it should list 'padding-block-end > 20px'.

Sebastian

[1] http://dev.w3.org/csswg/css-logical-props/

Comment 1

[Patrick Brosset <:pbro>]

Thanks for filing Sebastian, indeed we don't support this for now, but it'd be good to do.

Also weird with your test case, the padding-bottom property isn't displayed at all unless you check the "browser styles" checkbox.

It looks like supporting this would require some changes to css-logic.js/CssPropertyInfo which, right now, only knows about physical property name, and doesn't know how to map them back to corresponding logical properties.
We'll need a platform API (probably a new method in inIDOMUtils.idl) that provides this mapping, if there isn't one already.

Comment 2

[Patrick Brosset <:pbro>]

Filed bug 1132868.

Comment 3

[Julian Descottes [:jdescottes]]

Inspector bug triage. Filter on CLIMBING SHOES.

CSS logical properties are only supported on FF for now => P3.