1437890 - Move Layout view before the Computed view

Status: RESOLVED FIXED

(DevTools :: Inspector, enhancement, P3)

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: Bug 1437890 - Move Layout before Computed in Inspector.

Review commit: https://reviewboard.mozilla.org/r/221094/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/221094/

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=40b6fed758d693557f8f12f9c2a6b0f3d5edeb7e

Comment 3

[Gabriel Luong [:gl] (ΦωΦ)]

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.

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

We agreed in chat that it's okay to land this now by having it check the 3 Pane feature pref.

Comment 5

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8951806 [details]
Bug 1437890 - Move Layout before Computed in Inspector.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/221094/diff/1-2/

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8951806 [details]
Bug 1437890 - Move Layout before Computed in Inspector.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/221094/diff/2-3/

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c5709d4c6e4
Move Layout before Computed in Inspector. r=gl

Comment 8

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: Bug 1437890 - Use existing split pref value.

Review commit: https://reviewboard.mozilla.org/r/224066/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/224066/

Comment 9

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8954905 [details]
Bug 1437890 - Use existing split pref value.

https://reviewboard.mozilla.org/r/224066/#review230062

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c28537dcbda5
Use existing split pref value. r=jryans

Comment 11

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/8c5709d4c6e4
https://hg.mozilla.org/mozilla-central/rev/c28537dcbda5

Comment 12

[Saheda Reza [:Antora]]

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