Closed Bug 1496780 Opened 6 years ago Closed 6 years ago

Make the about:performance layout match the UX mockup

Categories

(Toolkit :: Performance Monitoring, defect)

Product:
Toolkit
Component:
Performance Monitoring
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla64
Tracking Flags:
Tracking Status
firefox64 --- fixed
firefox65 --- verified
firefox66 --- verified

People

(Reporter: florian, Assigned: florian)

References

Details

(Whiteboard: [perf-tools])

Attachments

(1 file)

Patch
4.86 KB, patch
Felipe
: review+
Details | Diff | Splinter Review
The mockup at https://mozilla.invisionapp.com/share/SDOE65KGU2Z#/screens/323554995 shows the table taking the whole content area. It also has more whitespace around the favicons than the current implementation.
Attached patch Patch — — Splinter Review
Most of the new CSS rules here are to ensure the table header remains at the top, while the table body can have a scrollbar if needed.
Attachment #9014863 - Flags: review?(felipc)
Whiteboard: [perf-tools]
Attachment #9014863 - Flags: review?(felipc) → review+
Pushed by florian@queze.net: https://hg.mozilla.org/integration/mozilla-inbound/rev/48f96feffcfe Make the about:performance layout match the UX mockup, r=felipe.
https://hg.mozilla.org/mozilla-central/rev/48f96feffcfe
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64

All the UX elements are according to the mock-up document. Verified this issue on the latest Nightly 66.0a1 (2019-01-14) and 65.0b10 (64-bit)on Windows 10x64, macOS 10.13 and Ubuntu 16.04x64

Status: RESOLVED → VERIFIED
status-firefox65: --- → verified
status-firefox66: --- → verified
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: