Make the Layout view the default sidebar tab

RESOLVED FIXED in Firefox 64

Status

enhancement
P3
normal
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: gl, Assigned: gl)

Tracking

(Depends on 1 bug)

unspecified
Firefox 64
Dependency tree / graph

Firefox Tracking Flags

(firefox64 fixed)

Details

Attachments

(1 attachment)

Currently, the computed view is the default sidebar tab in 3 pane mode. This is to make the layout view the default sidebar tab.
Assignee: nobody → gl
Status: NEW → ASSIGNED
Priority: -- → P3
Posted patch 1499483.patchSplinter Review
Attachment #9017740 - Flags: review?(pbrosset)
Comment on attachment 9017740 [details] [diff] [review]
1499483.patch

Review of attachment 9017740 [details] [diff] [review]:
-----------------------------------------------------------------

Code change looks good to me. But I don't remember the product discussion on this topic, so let me ping Victoria and Martin to provide some rationale behind this change for reference.
Attachment #9017740 - Flags: review?(pbrosset) → review+
Victoria, Martin, could you please take a look at comment 3?
Flags: needinfo?(victoria)
Flags: needinfo?(mbalfanz)
Yes, this would be great!

For me it's because: we've increased the importance of Layout by only showing the box model diagram there, are making a potentially very high-use tool with Flexbox, it's the future of integral Design Tools which have better debugging info that overlaps with (and is better organized) than Computed, and it's unique to our tools. Anyone who needs Computed can easily find it.

I may be overly biased by the fact that in 10 years of debugging websites, I only ever used Rules, not Computed, so that's a little weird.
Flags: needinfo?(victoria)
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0658fbb91e05
Make the Layout view the default sidebar tab. r=pbro
Backed out changeset 0658fbb91e05 (Bug 1499483) for devtools chrome failure at devtools/client/inspector/test/browser_inspector_sidebarstate.js on a CLOSED TREE

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&selectedJob=206229776&revision=0658fbb91e05b270f67252ec87e918d59d4aaf6d

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=206245623&repo=mozilla-inbound&lineNumber=4165
            https://treeherder.mozilla.org/logviewer.html#?job_id=206229776&repo=mozilla-inbound&lineNumber=19538

Backout link: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&selectedJob=206229776&revision=0d9b7b7a5aa5d30412233ffbbabe24dd26df97d0

[task 2018-10-17T22:40:24.617Z] 22:40:24     INFO - Buffered messages finished
[task 2018-10-17T22:40:24.618Z] 22:40:24     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_sidebarstate.js | object is correct - Got layoutview, expected computedview
[task 2018-10-17T22:40:24.619Z] 22:40:24     INFO - Stack trace:
[task 2018-10-17T22:40:24.620Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:test_is:1295
[task 2018-10-17T22:40:24.621Z] 22:40:24     INFO - chrome://mochitests/content/browser/devtools/client/inspector/test/browser_inspector_sidebarstate.js:checkTelemetryResults:97
[task 2018-10-17T22:40:24.622Z] 22:40:24     INFO - chrome://mochitests/content/browser/devtools/client/inspector/test/browser_inspector_sidebarstate.js:null:79
[task 2018-10-17T22:40:24.623Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1093
[task 2018-10-17T22:40:24.624Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1084
[task 2018-10-17T22:40:24.625Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:986
[task 2018-10-17T22:40:24.626Z] 22:40:24     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
[task 2018-10-17T22:40:24.627Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | value is correct - 
[task 2018-10-17T22:40:24.630Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | timestamp is greater than 0 - 
[task 2018-10-17T22:40:24.631Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | time_open is greater than 0 - 
[task 2018-10-17T22:40:24.635Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | category is correct - 
[task 2018-10-17T22:40:24.637Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | method is correct - 
[task 2018-10-17T22:40:24.638Z] 22:40:24     INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-10-17T22:40:24.639Z] 22:40:24     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_sidebarstate.js | object is correct - Got computedview, expected layoutview
[task 2018-10-17T22:40:24.640Z] 22:40:24     INFO - Stack trace:
[task 2018-10-17T22:40:24.642Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:test_is:1295
[task 2018-10-17T22:40:24.643Z] 22:40:24     INFO - chrome://mochitests/content/browser/devtools/client/inspector/test/browser_inspector_sidebarstate.js:checkTelemetryResults:97
[task 2018-10-17T22:40:24.644Z] 22:40:24     INFO - chrome://mochitests/content/browser/devtools/client/inspector/test/browser_inspector_sidebarstate.js:null:79
[task 2018-10-17T22:40:24.645Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1093
[task 2018-10-17T22:40:24.646Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1084
[task 2018-10-17T22:40:24.647Z] 22:40:24     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:986
[task 2018-10-17T22:40:24.649Z] 22:40:24     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
[task 2018-10-17T22:40:24.650Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | value is correct - 
[task 2018-10-17T22:40:24.651Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | timestamp is greater than 0 - 
[task 2018-10-17T22:40:24.653Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | time_open is greater than 0 - 
[task 2018-10-17T22:40:24.654Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | category is correct - 
[task 2018-10-17T22:40:24.655Z] 22:40:24     INFO - TEST-PASS | devtools/client/inspector/test/browser_inspector_sidebarstate.js | method is correct - 
[task 2018-10-17T22:40:24.656Z] 22:40:24     INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-10-17T22:40:24.658Z] 22:40:24     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_sidebarstate.js | object is correct - Got layoutview, expected ruleview
Flags: needinfo?(gl)
Flags: needinfo?(gl)
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1dc2238f2ba7
Make the Layout view the default sidebar tab. r=pbro
https://hg.mozilla.org/mozilla-central/rev/1dc2238f2ba7
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
I agree with Victoria (comment 5), so good to go from my point of view.
Flags: needinfo?(mbalfanz)
Flags: needinfo?(gl)
You need to log in before you can comment on or make changes to this bug.