Closed
Bug 1913693
Opened 2 months ago
Closed 1 month ago
Remove legacy code to handling escaping out of the debugger editor which is causing issues
Categories
(DevTools :: Debugger, defect, P3)
DevTools
Debugger
Tracking
(firefox131 fixed)
RESOLVED
FIXED
131 Branch
| Tracking | Status | |
|---|---|---|
| firefox131 | --- | fixed |
People
(Reporter: bomsy, Assigned: bomsy)
References
(Blocks 3 open bugs)
Details
Attachments
(1 file)
This is some legacy code probably added to fix an issue which seems to no longer exist. See https://searchfox.org/mozilla-central/rev/2455e4d6388cbdeadb44b9633b971a65a98b504c/devtools/client/debugger/src/components/Editor/index.js#472-487. One effect of this is that we need to hit Esc twice to open the split console, when focused in the debugger. But we should only have to hit esc once.
STR
- Open any source in the debugger
- Click in the editor at any position
- Press Esc
Expected Result
The split console should open
Actual Result
Nothing happens (Needs a second escape to open the split console)
|
Assignee | |
Updated•2 months ago
|
Summary: Remove legacy code to handle escaping out of the debugger editor → Remove legacy code to handling escaping out of the debugger editor which is causing issues
|
|
Assignee | |
Comment 1•2 months ago
|
||
:
|
||
Updated•2 months ago
|
Assignee: nobody → hmanilla
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•2 months ago
|
Severity: -- → S3
Priority: -- → P3
|
|
Assignee | |
Updated•1 month ago
|
Blocks: devtools-a11y
Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c88bf4be8ae7
[devtools] Cleanup legacy code added to handle escaping out of the editor r=devtools-reviewers,jdescottes
|
||
Comment 3•1 month ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
status-firefox131:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch
|
||
Updated•1 month ago
|
QA Whiteboard: [qa-131b-p2]
You need to log in
before you can comment on or make changes to this bug.
Description
•