Polish the box model properties

RESOLVED FIXED in Firefox 55

Status

P3
normal
RESOLVED FIXED
2 years ago
10 months ago

People

(Reporter: gl, Assigned: lockhart)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 55
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Currently, the box model properties are styled exactly like the computed view, but given the space we have and the fact that it doesn't work like the computed view, we should make sure they don't overflow and fold into the next line.
(Reporter)

Updated

2 years ago
Priority: -- → P3
(Reporter)

Updated

2 years ago
Assignee: nobody → lockhart
Status: NEW → ASSIGNED
(Reporter)

Comment 1

2 years ago
I think we should also change .boxmodel-position.boxmodel-top, .boxmodel-position.boxmodel-bottom to calc(50% - 2px) while we are at it. This seems more centered than - 1px.
Comment hidden (mozreview-request)
(Reporter)

Comment 3

2 years ago
mozreview-review
Comment on attachment 8852249 [details]
Bug 1349686: Polish the Box Model CSS.

https://reviewboard.mozilla.org/r/124474/#review127342

Give my suggestions a try and let me know what you think.

::: devtools/client/themes/boxmodel.css:328
(Diff revision 1)
>  .boxmodel-properties-wrapper {
>    padding: 0 9px;
>  }
>  
> +.boxmodel-properties-wrapper .property-name-container {
> +  width: 50%;

I think we can just do "flex: 1" here.

::: devtools/client/themes/boxmodel.css:332
(Diff revision 1)
> +.boxmodel-properties-wrapper .property-name-container {
> +  width: 50%;
> +}
> +
> +.boxmodel-properties-wrapper .property-value-container {
> +  width: 50%;

And "flex: 1" here as well.
Attachment #8852249 - Flags: review?(gl)
Comment hidden (mozreview-request)
(Reporter)

Comment 5

2 years ago
mozreview-review
Comment on attachment 8852249 [details]
Bug 1349686: Polish the Box Model CSS.

https://reviewboard.mozilla.org/r/124474/#review127474
Attachment #8852249 - Flags: review?(gl) → review+

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8d2b523a1a71
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
I have successfully reproduced this issue on firefox nightly according to(2017-03-22)

Fixing bug is verified on Latest Nightly-- Build ID: (20170402030202),User Agent: Mozilla/5.0 (Windows NT 10.0; rv:55.0) Gecko/20100101 Firefox/55.0

Tested OS -- Windows10 32bit

[bugday-20170329]

Updated

10 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.