Closed Bug 1437890 Opened 2 years ago Closed 2 years ago

Move Layout view before the Computed view

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(firefox60 fixed)

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: gl, Assigned: jryans)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

No description provided.
Component: Developer Tools: Computed Styles Inspector → Developer Tools: Inspector
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Comment on attachment 8951806 [details]
Bug 1437890 - Move Layout before Computed in Inspector.

https://reviewboard.mozilla.org/r/221094/#review227962

I don't think this should land until we enable three-pane.
Attachment #8951806 - Flags: review?(gl) → review+
We agreed in chat that it's okay to land this now by having it check the 3 Pane feature pref.
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/8c5709d4c6e4
Move Layout before Computed in Inspector. r=gl
Comment on attachment 8954905 [details]
Bug 1437890 - Use existing split pref value.

https://reviewboard.mozilla.org/r/224066/#review230062
Attachment #8954905 - Flags: review+
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/c28537dcbda5
Use existing split pref value. r=jryans
https://hg.mozilla.org/mozilla-central/rev/8c5709d4c6e4
https://hg.mozilla.org/mozilla-central/rev/c28537dcbda5
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
QA Whiteboard: [good first verify]
This request is implemented on Latest Nightly. 

Layout view has been moved before computer view .

Fixing bug is verified on latest Nightly--
Build ID    :20180421220102
User Agent  :Mozilla/5.0 (Windows NT 6.1; WOW64; rv:61.0) Gecko/20100101 Firefox/61.0

Tested OS-- Windows7 64bit
QA Whiteboard: [good first verify] → [testday-20180420]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.