Closed
Bug 1219801
Opened 9 years ago
Closed 9 years ago
The toolbar in the memory panel on Windows is not looking too nice
Categories
(DevTools :: Memory, defect)
Tracking
(firefox44 affected, firefox45 fixed)
RESOLVED
FIXED
Firefox 45
People
(Reporter: pbro, Assigned: vporof)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
In particular, the select and input elements aren't properly vertically aligned with their labels.
Reporter | ||
Updated•9 years ago
|
OS: Unspecified → Windows 10
Updated•9 years ago
|
Blocks: memory-tools-fx44
Reporter | ||
Comment 1•9 years ago
|
||
Things have a moved around a little bit over the week end. I just updated my fx-team repo and here's what I'm seeing.
The vertical-alignment of labels is still off a bit. And now the drop-down is a bit too high.
Apparently the <div> that contains the "group by", "invert", "stacks" controls is absolutely positioned now, while it was flex before. This seems to make harder to position correctly. I think it would work better in flex.
I think it was changed to absolute so that it would start only at var(--sidebar-width) pixels from the left side.
I think this would work better:
.devtools-toolbar .devtools-button.take-snapshot {
-moz-appearance: none;
margin-inline-start: 1px;
margin-inline-end: calc(var(--sidebar-width) - 32px);
}
.devtools-toolbar .toolbar-group {
display: flex;
}
.toolbar-group > label {
margin-inline-end: 5px;
display: flex;
align-items: center;
}
Assignee | ||
Comment 2•9 years ago
|
||
Reporter | ||
Comment 3•9 years ago
|
||
With the last patch in bug 1213100, here's what the toolbar looks like.
It's a little better, but we still need better vertical alignment.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → vporof
Status: NEW → ASSIGNED
QA Contact: vporof
Assignee | ||
Comment 5•9 years ago
|
||
Attachment #8682529 -
Flags: review?(bgrinstead)
Updated•9 years ago
|
No longer blocks: memory-tools-fx44
Comment 6•9 years ago
|
||
Comment on attachment 8682529 [details] [diff] [review]
v1
Review of attachment 8682529 [details] [diff] [review]:
-----------------------------------------------------------------
This fixes the alignment issues, thanks! Please update the commit message to be more descriptive of the fix.
I keep accidentally selecting the label text when double clicking on a label so maybe we want to do some -moz-user-select magic there, but we could do that in another bug
Attachment #8682529 -
Flags: review?(bgrinstead) → review+
Updated•9 years ago
|
Blocks: memory-frontend
Updated•9 years ago
|
Has STR: --- → yes
Comment 8•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Updated•9 years ago
|
status-firefox44:
--- → affected
Updated•6 years ago
|
Product: Firefox → DevTools
Assignee | ||
Updated•6 years ago
|
QA Contact: vporof
You need to log in
before you can comment on or make changes to this bug.
Description
•