Closed Bug 1559258 Opened 1 year ago Closed 1 year ago

[Protections Panel] Clean up margins and padding in all panel sections

Categories

(Firefox :: Site Identity, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 70
Tracking Status
firefox69 --- wontfix
firefox70 --- verified

People

(Reporter: nhnt11, Assigned: nhnt11)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

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

Attachments

(2 files)

For example, the labels in the toggle section have margins, based on which the padding is set for the section.

We should clean all this up by removing padding/margin on individual elements, and setting consistent padding on the boxes.

Whiteboard: [privacy-panel][triage]
Priority: -- → P1
Whiteboard: [privacy-panel][triage] → [privacy-panel]
Whiteboard: [privacy-panel] → [privacy-panel] [protection-report] [skyline]
Depends on: 1567296
No longer depends on: 1567296
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Attachment #9083823 - Attachment mime type: text/plain → video/webm
Attachment #9083820 - Attachment description: Bug 1559258 - Clean up margins and paddings in Protections Panel, increase font-size to spec. r=timhuang → Bug 1559258 - Clean up margins, paddings, and font-sizes in Protections Panel. r=dao
Attachment #9083820 - Attachment description: Bug 1559258 - Clean up margins, paddings, and font-sizes in Protections Panel. r=dao → Bug 1559258 - Clean up margins, paddings, and font-sizes in Protections Panel. r=johannh
Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/2b29ae6ba20f
Clean up margins, paddings, and font-sizes in Protections Panel. r=johannh
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Regressed by: 1577257
No longer regressed by: 1577257
Regressions: 1577257
Regressions: 1578187
Flags: qe-verify+

From the description on the first comment I've seen that in the 69.0a1 (2019-06-13) the UI in the Privacy Panel has the margins/paddings visible.
I can confirm that the UI corresponds to the description in the comment 2 and is fixed in 70.0b6.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.