Enable editor mode everywhere
Categories
(DevTools :: Console, task, P2)
Tracking
(firefox71+ verified, firefox72 verified)
People
(Reporter: nchevobbe, Assigned: nchevobbe)
References
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Since we added a deprecation warning in Scratchpad in 70, and since we talk about the editor mode as an alternative in the MDN page we link to from the deprecation message, I wonder if we could enable the editor mode in Beta for 70.
Comment 2•5 years ago
|
||
Yes, we should get editor mode roll into release unless any blockers get filed. So far I am not aware of any.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
The focus previous element function needed to be modified
in order to not find non-visible elements.
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c8a702f3b356 Enable editor mode everywhere. r=Honza.
Comment 5•5 years ago
|
||
bugherder |
Assignee | ||
Comment 6•5 years ago
|
||
Comment on attachment 9102523 [details]
Bug 1576568 - Enable editor mode everywhere. r=Honza.
Beta/Release Uplift Approval Request
- User impact if declined: Users won't have access to the new editor mode in the console.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: On any page
- Open the webconsole
- There should be an icon on the end of the input
- Click on it
it should put the input on the left side of the panel, taking all the vertical space and adding a toolbar
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The editor mode was already in DevEdition since 70.
- String changes made/needed:
Assignee | ||
Updated•5 years ago
|
Comment 7•5 years ago
|
||
Comment on attachment 9102523 [details]
Bug 1576568 - Enable editor mode everywhere. r=Honza.
Fix for a visible bug in devtools, patch with tests, uplift approved for 71 beta 4, let's have QA verify the fix in beta with the STR provided in the uplift request. Thanks!
Updated•5 years ago
|
Updated•5 years ago
|
Comment 8•5 years ago
|
||
Hello!
Using STR from comment 6 with Firefox 72.0a1 (20191022214314) the editor is correctly open as described on Windows 10x64, macOS 10.14 and Ubuntu 18.04.
Comment 9•5 years ago
|
||
bugherder uplift |
Comment 10•5 years ago
•
|
||
Description
•