DevTools Performance tab - spacing polish
Categories
(DevTools :: Performance Tools (Profiler/Timeline), enhancement, P3)
Tracking
(firefox93 fixed)
| Tracking | Status | |
|---|---|---|
| firefox93 | --- | fixed |
People
(Reporter: julienw, Assigned: jdescottes, Mentored)
References
Details
Attachments
(3 files)
image.png
This is copied over from https://github.com/firefox-devtools/profiler/issues/2640.
From victoria:
We should have less top spacing when DevTools is short. For the Accessibility welcome panel (in stable only) the semi-responsive way we did this was padding-top: 15vh;. For the Performance tab I might suggest
.perf-devtools .perf-button-container {
margin-top: 10vh;
}
When DevTools is docked to side, we should have padding of 10vh on each side. Also, we can have the 'Settings' label be on its own line. Open the attachment to see how this could look:
|
Reporter | |
Updated•4 years ago
|
|
||
Comment 2•4 years ago
|
||
Depends on D100588
|
||
Updated•4 years ago
|
|
Assignee | |
Comment 3•3 years ago
|
||
Would be nice to fix this before enabling on release channel.
Julien, can you take a look at the review request?
|
|
Assignee | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 5•3 years ago
|
||
Hi kaeyosthaeron and sorry for the delay!
I requested a few changes on the patch.
Considering it is 8 months old, I want to check if you are still interested in finishing this work.
Let us know!
|
|
Assignee | |
Comment 6•3 years ago
|
||
No answer, will assume we can pick it up from here.
|
|
Assignee | |
Comment 7•3 years ago
|
||
|
|
Assignee | |
Updated•3 years ago
|
|
||
Comment 9•3 years ago
|
||
Thanks for working on this, Julian :D
|
||
Comment 10•3 years ago
|
||
| bugherder | ||
Description
•