Closed
Bug 1390298
Opened 7 years ago
Closed 7 years ago
Change the fonts used in the Inspector
Categories
(DevTools :: Inspector, defect, P3)
DevTools
Inspector
Tracking
(firefox57 fixed)
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: gl, Assigned: gl)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
10.24 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
Inspector side panel: Section headings shouldn’t be monospace - use system font instead for everything in the gutters/accordion, headers/secondary toolbars and including the following: Rules: "Inherited from body" Computed: “Browser styles,” "Box Model” Layout: “Grid,” “Box Model,” “Box Model Properties” Animations: “All animations” Fonts: “show all fonts used” Informational (non-code) text like on Layout, Animations, and Fonts also should be system font instead of monospace
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Attachment #8897572 -
Attachment is obsolete: true
Attachment #8897572 -
Flags: review?(bgrinstead)
Attachment #8897604 -
Flags: review?(bgrinstead)
Updated•7 years ago
|
Attachment #8897604 -
Flags: review?(bgrinstead) → review+
Pushed by gabriel.luong@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d3b1275a43bf Change the fonts used in the Inspector. r=bgrins
Pushed by gabriel.luong@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d1d85eed4d06 Fix unit tests with browser_boxmodel_navigation.js not recognizing ".boxmodel-main .devtools-monospace". r=me
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d3b1275a43bf https://hg.mozilla.org/mozilla-central/rev/d1d85eed4d06
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•