Closed
Bug 1037145
Opened 10 years ago
Closed 10 years ago
Remove esc as a keybinding to close responsive mode
Categories
(DevTools :: Responsive Design Mode, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 34
People
(Reporter: canuckistani, Assigned: bgrins)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
paul
:
review+
|
Details | Diff | Splinter Review |
Having a global binding on the escape key to close responsive mode is causing lots of problems with other places in the code where we bind escape, see this thread for more discussion: https://groups.google.com/forum/#!topic/mozilla.dev.developer-tools/icxiZWPI3tw Copying in cases where this trips us up: Scenario 1: 1. open a site 2. enable responsive design view 3. open the toolbox and switch to the debugger by clicking on the tool tab or using cmd+shift+] 4. hit esc Result: responsive design view toggles off Expected: split console opens, responsive design view stays on Scenario 2: 1. open a site 2. enable responsive design view 3. open the toolbox and switch to the debugger 4. open the split console using the toolbox button 5. type in 'window' and hit enter, then click on window 6. hit escape 3 times Result: variable viewer closes, then responsive design view, then the split console Expected: variable viewer closes, then the split console, then responsive design view
Assignee | ||
Comment 2•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=5ae179f420a8
Attachment #8461068 -
Attachment is obsolete: true
Attachment #8461578 -
Flags: review?(paul)
Updated•10 years ago
|
Attachment #8461578 -
Flags: review?(paul) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/c74f359e58d2
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/c74f359e58d2
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 34
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•