Fix the styling of multitable section in about:telemetry

RESOLVED FIXED in Firefox 57

Status

()

defect
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: flyingrub, Assigned: flyingrub)

Tracking

(Blocks 1 bug)

unspecified
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

2 years ago
The headings in multi-table sections are oddly-sized (e.g. environment -> addons). The "activeAddons" heading is smaller than the following table heading / addon name. It is hard to separate from the rest of the sections as there is not enough space to the preceding element.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Attachment #8895999 - Flags: review?(gfritzsche) → review?(chutten)

Comment 3

2 years ago
mozreview-review
Comment on attachment 8895999 [details]
Bug 1389255 - Fix the styling of multitable section in about:telemetry

https://reviewboard.mozilla.org/r/167272/#review173610

::: toolkit/content/aboutTelemetry.css:300
(Diff revision 2)
>  caption {
>    font-weight: bold;
>    white-space: nowrap;
>    text-align: left;
> -  font-size: large;
> +  font-size: 24px;
> +  margin: 24px 0 12px 0;

Can we get any of these values out of Common.css? Or peg them to the current font size?
Attachment #8895999 - Flags: review?(chutten)
Assignee

Comment 4

2 years ago
mozreview-review
Comment on attachment 8895999 [details]
Bug 1389255 - Fix the styling of multitable section in about:telemetry

https://reviewboard.mozilla.org/r/167272/#review173996

::: toolkit/content/aboutTelemetry.css:300
(Diff revision 2)
>  caption {
>    font-weight: bold;
>    white-space: nowrap;
>    text-align: left;
> -  font-size: large;
> +  font-size: 24px;
> +  margin: 24px 0 12px 0;

I don't think we can get them from common.css.
I cleaned it up a little and use em instead of px now.
Comment hidden (mozreview-request)
Assignee

Comment 6

2 years ago
mozreview-review-reply
Comment on attachment 8895999 [details]
Bug 1389255 - Fix the styling of multitable section in about:telemetry

https://reviewboard.mozilla.org/r/167272/#review173610

> Can we get any of these values out of Common.css? Or peg them to the current font size?

I switched to em instead of px. Also useless css removed.
Attachment #8895999 - Flags: review?(gfritzsche) → review?(chutten)

Comment 7

2 years ago
mozreview-review
Comment on attachment 8895999 [details]
Bug 1389255 - Fix the styling of multitable section in about:telemetry

https://reviewboard.mozilla.org/r/167272/#review174036
Attachment #8895999 - Flags: review?(chutten) → review+
Assignee

Updated

2 years ago
Keywords: checkin-needed

Comment 8

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/cca3c23a2235
Fix the styling of multitable section in about:telemetry r=chutten
Keywords: checkin-needed

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cca3c23a2235
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.