Closed
Bug 1521612
Opened 7 years ago
Closed 7 years ago
Add pref for combined flexbox highlighter
Categories
(DevTools :: Inspector, enhancement, P2)
DevTools
Inspector
Tracking
(firefox66 fixed)
RESOLVED
FIXED
Firefox 66
| Tracking | Status | |
|---|---|---|
| firefox66 | --- | fixed |
People
(Reporter: miker, Assigned: miker)
Details
Attachments
(1 file)
The combined flexbox highlighter should be disabled if either of the following prefs are false:
devtools.inspector.flexboxHighlighter.enableddevtools.inspector.flexboxHighlighter.combine
devtools.inspector.flexboxHighlighter.combine should be true in dev edition and false in release.
|
Assignee | |
Comment 1•7 years ago
|
||
Because checking for prefs is slow and performance is paramount for the highlighter we cache the result. Because we cache the result the Toolbox needs to be closed and opened again for any pref changes to take affect.
Pushed by mratcliffe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b038cb88e341
Add pref for combined flexbox highlighter r=pbro
|
||
Comment 3•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
|
||
Comment 4•7 years ago
|
||
https://hg.mozilla.org/projects/cedar/rev/b038cb88e3416f0de01204590f772dbf2d47d2f9
Bug 1521612 - Add pref for combined flexbox highlighter r=pbro
|
||
Updated•7 years ago
|
Assignee: nobody → mratcliffe
|
||
Updated•7 years ago
|
Flags: qe-verify+
QA Contact: cristian.comorasu
|
|
||
Updated•7 years ago
|
Flags: qe-verify+
QA Contact: cristian.comorasu
You need to log in
before you can comment on or make changes to this bug.
Description
•