1348258 - ElementNode Rep in the CSS Grid layout panel should not show all attributes

Status: RESOLVED FIXED

(DevTools :: Inspector, defect, P3)

Description

[Patrick Brosset <:pbro>]

STR:
- open this URL
data:text/html,<div test="test" style="display:grid;grid-template-rows:repeat(50, 100px);grid-template-columns:repeat(20, 100px);">
- open the css grid panel in the inspector sidebar

=> the div that has the grid defined is listed there, and previewed using an ElementNode Rep.

Right now, this Rep shows all attributes, but I don't think that's useful, and it takes a lot of space.
I think it should display nodes just like the animation inspector does, like:
tag#id.class

Comment 1

[Patrick Brosset <:pbro>]

Attachment: Bug 1348258 - Use MODE.TINY for ElementNodes displayed in the grid inspector;

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

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8848456 [details]
Bug 1348258 - Use MODE.TINY for ElementNodes displayed in the grid inspector;

https://reviewboard.mozilla.org/r/121372/#review123410

Looks good !

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bb5c698bff0a
Use MODE.TINY for ElementNodes displayed in the grid inspector; r=nchevobbe

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/bb5c698bff0a