Closed Bug 1570624 Opened 5 years ago Closed 5 years ago

Fix font-size values in the protections panel

Categories

(Firefox :: Protections UI, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: johannh, Assigned: timhuang)

References

Details

(Whiteboard: [privacy-panel][skyline])

Attachments

(1 file)

In the spec all the fonts have the same size, while in the real protection panel I feel like some text is smaller than others. They might be missing from the infamous 110% rule.

We should go through it and add selectors as appropriate.

(In reply to Johann Hofmann [:johannh] from comment #0)

In the spec all the fonts have the same size, while in the real protection panel I feel like some text is smaller than others. They might be missing from the infamous 110% rule.

We should go through it and add selectors as appropriate.

Why would you try to target every single element rather than setting the font-size on a common container?

I can confirm that the font-sizes are inconsistent, it's not just your feeling.

Type: task → defect
Summary: Audit font-size values in the protections panel → Fix font-size values in the protections panel

It would be nice to have a common container font size but I tried this once a while ago and it broke sizing of, uh, things (I don't exactly remember), so I abandoned it. Maybe we'll have better luck this time. Certainly worth a try!

Priority: -- → P1
Assignee: nobody → tihuang
Status: NEW → ASSIGNED

This patch adds a css rule 'font-size' to the protection panel in order
to set size to the all strings in the protection panel.

Pushed by tihuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79ee8db4c841
Set the font-size to the protection panel. r=nhnt11,dao
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: