Remove the megabar's "focused & collapsed" state
Categories
(Firefox :: Address Bar, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: dao, Assigned: dao)
References
()
Details
Attachments
(1 file)
The "collapsed" state was introduced so the user could interact with elements that could otherwise be overlapped by the megabar. Now we that we don't overlap anymore, this state seems unnecessary.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
If you remove the collapsed state (which I find a good solution since this contast size changing is driving me crazy), are you going to keep the shadow? Or will the megabar only cast a shadow if it is focused?
And how would you show that the megabar is focused? With a blue border?
Comment 3•5 years ago
|
||
(In reply to trinywiky from comment #2)
And how would you show that the megabar is focused? With a blue border?
Removing the collapsed state means that it will be more obvious that the megabar is focused: if it is focused, it is in its expanded state. It will also retain its border.
Dao's patch still applies cleanly to central and meets all four bullet-point criteria for Megabar states in the new spec.
Comment 5•5 years ago
|
||
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=274620281&repo=autoland
Backout link: https://hg.mozilla.org/integration/autoland/rev/565c9ca9735cff29f5de9b462ad00da331387107
[task 2019-11-05T13:45:24.645Z] 13:45:24 INFO - TEST-PASS | browser/components/urlbar/tests/browser/browser_urlbar_collapseOnChromeMousedown.js | The Urlbar should have the breakout-extend attribute. - true == true -
[task 2019-11-05T13:45:24.646Z] 13:45:24 INFO - Buffered messages finished
[task 2019-11-05T13:45:24.647Z] 13:45:24 INFO - TEST-UNEXPECTED-FAIL | browser/components/urlbar/tests/browser/browser_urlbar_collapseOnChromeMousedown.js | The Urlbar should not have the breakout-extend attribute. - false == true - JS frame :: chrome://mochitests/content/browser/browser/components/urlbar/tests/browser/browser_urlbar_collapseOnChromeMousedown.js :: <TOP_LEVEL> :: line 34
[task 2019-11-05T13:45:24.647Z] 13:45:24 INFO - Stack trace:
[task 2019-11-05T13:45:24.647Z] 13:45:24 INFO - chrome://mochitests/content/browser/browser/components/urlbar/tests/browser/browser_urlbar_collapseOnChromeMousedown.js:null:34
[task 2019-11-05T13:45:24.647Z] 13:45:24 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1069
[task 2019-11-05T13:45:24.647Z] 13:45:24 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1104
[task 2019-11-05T13:45:24.648Z] 13:45:24 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:932
[task 2019-11-05T13:45:24.651Z] 13:45:24 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:805
[task 2019-11-05T13:45:24.651Z] 13:45:24 INFO - TEST-PASS | browser/components/urlbar/tests/browser/browser_urlbar_collapseOnChromeMousedown.js | The Urlbar should be focused. - true == true -
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Dao is OoO so I'm going to see this through to landing. The failing test tested the behaviour being removed so the test can be removed. I've updated the revision on Phabricator.
Updated•5 years ago
|
Comment 8•5 years ago
|
||
bugherder |
Description
•