Closed Bug 689924 Opened 13 years ago Closed 13 years ago

Change "Inspect" shortcut to Cmd+Opt+I, Web Console to Cmd+Opt+K (Mac Only)

Categories

(DevTools :: General, defect)

All
macOS
defect
Not set
normal

Tracking

(firefox10+ verified)

VERIFIED FIXED
Firefox 11
Tracking Status
firefox10 + verified

People

(Reporter: rik, Unassigned)

Details

(Keywords: verified-beta, Whiteboard: [fixed-in-fx-team][qa!])

Attachments

(1 file, 1 obsolete file)

Safari, Chrome and Opera uses Cmd+Opt+I to launch their devtools. I think we should match that. Might be worth changing other shortcuts to replace Shift by Opt also.
> Might be worth changing other shortcuts to replace Shift by Opt also. Which ones?
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
I'm gonna reopen this one because it's only about Mac shortcuts. The other bug says it can't be done because of Windows bindings. (In reply to Paul Rouget [:paul] from comment #1) > > Might be worth changing other shortcuts to replace Shift by Opt also. > > Which ones? Web Console and Error Console currently use Shift.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Attached patch Patch (obsolete) — Splinter Review
This changes the Web Console and Inspect tool to use Cmd+Alt shortcuts. I left out the Error console cause I think it's not gonna be a tool for web developers.
Assignee: nobody → anthony
Status: REOPENED → ASSIGNED
Attachment #574111 - Flags: review?(dcamp)
Comment on attachment 574111 [details] [diff] [review] Patch >+#ifdef XP_MACOSX >+ <key id="key_webConsole" key="&webConsoleCmd.commandkey;" oncommand="HUDConsoleUI.toggleHUD();" modifiers="accel,alt"/> >+#else > <key id="key_webConsole" key="&webConsoleCmd.commandkey;" oncommand="HUDConsoleUI.toggleHUD();" modifiers="accel,shift"/> >+#endif <key id="key_webConsole" key="&webConsoleCmd.commandkey;" #ifdef XP_MACOSX modifiers="accel,alt" #else modifiers="accel,shift" #endif oncommand="HUDConsoleUI.toggleHUD();"/>
Attached patch Patch v2Splinter Review
Thanks for the comment Dao, fixed. I've put Dave for the review but I have no idea who I should ask to review.
Attachment #574200 - Flags: review?(dcamp)
(In reply to Anthony Ricaud (:rik) from comment #6) > Created attachment 574200 [details] [diff] [review] [diff] [details] [review] > Patch v2 > > Thanks for the comment Dao, fixed. > > I've put Dave for the review but I have no idea who I should ask to review. You could ask any of the fine gentlemen listed in this page: https://wiki.mozilla.org/Modules/Firefox
Attachment #574111 - Attachment is obsolete: true
Attachment #574111 - Flags: review?(dcamp)
Comment on attachment 574200 [details] [diff] [review] Patch v2 Thanks Panos. Then I guess I should ask Dao.
Attachment #574200 - Flags: review?(dcamp) → review?(dao)
Attachment #574200 - Flags: review?(dao) → review+
Keywords: checkin-needed
Updating the summary to reflect reality. This may severely impact people's muscle memory.
Summary: Change "Inspect" shortcut to Cmd+Opt+I → Change "Inspect" shortcut to Cmd+Opt+I, Web Console to Cmd+Opt+K (Mac Only)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
Comment on attachment 574200 [details] [diff] [review] Patch v2 Since this changes the key for a new feature, we should land this in aurora to get people used to it. The webconsole key-binding should be considered a companion setting. Low-risk. No code changes.
Attachment #574200 - Flags: approval-mozilla-aurora?
[triage comment] Is there any downside to supporting both the current and this shortcut for web console? Hasn't web console shipped with that shortcut in multiple releases, and wouldn't we be breaking muscle memory that way?
(In reply to Christian Legnitto [:LegNeato] from comment #13) > [triage comment] > Is there any downside to supporting both the current and this shortcut for > web console? Hasn't web console shipped with that shortcut in multiple > releases, and wouldn't we be breaking muscle memory that way? like, having two shortcuts assigned for the web console? Only downside is some untested xul. It does break muscle memory, and in some cases, interferes with other shortcuts requiring users to reconfigure hotkeys outside of Firefox (in my case, OmniFocus).
Attachment #574200 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Did you test that this doesn't break existing localized shortcuts in localizations? (I don't think we have anything better than tweaking a mozmill l10n test and run through all localized builds.)
uh, no, I have no idea how to test that. I filed a follow-up bug to this: bug 706204 to restore the original web console shortcut as per legneato's comment 13. I'd like to get that into aurora at the same time.
I just received an email asking me to land this on Aurora. Since I cannot do that, de-assigning myself. Can someone take it?
Assignee: anthony → nobody
I will land that tomorrow.
Whiteboard: [fixed-in-fx-team] → [fixed-in-fx-team][qa+]
I have tried this on: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:10.0) Gecko/20100101 Firefox/10.0 beta 2 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0) Gecko/20100101 Firefox/10.0 beta 2 Cmd+Opt+I opens "Inspect" and Cmd+Opt+K opens "Web Console" Setting resolution to Verified Fixed
Status: RESOLVED → VERIFIED
Keywords: verified-beta
Whiteboard: [fixed-in-fx-team][qa+] → [fixed-in-fx-team][qa!]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: