Closed
Bug 1488883
Opened 6 years ago
Closed 6 years ago
DevTools flexbox inspector probably shouldn't capitalize "Max-width" and "Min-width" (and maybe "Content/Final width" too)
Categories
(DevTools :: Inspector, defect, P3)
DevTools
Inspector
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: dholbert, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
16.79 KB,
image/png
|
Details |
STR:
0. Set pref devtools.flexboxinspector.enabled to true
1. Visit this data URL:
data:text/html,<div style="display:flex"><span>abc
...or:
data:text/html,<div style="display:flex;flex-direction:column"><span>abc
2. Right-click "abc" and choose "inspect"
3. Click the "flex" badge in the DOM view.
4. In the "Layout" inspector pane (to the right), click "1. span" to expand the flex-item info for the <span>.
ACTUAL RESULTS:
Some of the labels are capitalized, some are not.
- "flex-grow", "flex-shrink", and "flex-basis" are lowercase.
- "Content width", "Min-width", "Max-width", "Final width" are capitalized
EXPECTED RESULTS:
Consistent capitalization (or lack thereof, really).
- At least "Min-width" and "Max-width" should be lower case since they're property names (which are case-insensitive but typically written as lowercase).
- For the other two, I think lowercase "content width" and "final width" feel cleaner/more-consistent than their capitalized counterparts, but this matters less since they're prose rather than property-names.
Reporter | ||
Updated•6 years ago
|
Summary: DevTools flexbox inspector probably shouldn't capitalize "Max-width" and "Min-width" → DevTools flexbox inspector probably shouldn't capitalize "Max-width" and "Min-width" (and maybe "Content/Final width" too)
Reporter | ||
Comment 1•6 years ago
|
||
Updated•6 years ago
|
Comment 2•6 years ago
|
||
This will become obsolete with bug 1495717 where we're re-working the appearance and content of this section of UI.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•