Open
Bug 51125
Opened 24 years ago
Updated 2 years ago
Accel-Comma should open Preferences (cmd-, on macos, ctrl-, on win32/linux)
Categories
(Core :: DOM: UI Events & Focus Handling, enhancement, P4)
Core
DOM: UI Events & Focus Handling
Tracking
()
NEW
mozilla1.5beta
People
(Reporter: mozilla7, Unassigned)
References
Details
(Keywords: polish, Whiteboard: [need r/sr])
Attachments
(1 file, 2 obsolete files)
5.52 KB,
patch
|
Details | Diff | Splinter Review |
Many Mac OS apps (including Internet Explorer) use Command-Semicolon as a shortcut to the Preferences dialog. Mozilla probably should as well. This is a Mac OS standard, although it should be implemented on all platforms for consistency. To my knowledge, no other platform has a standard shortcut for Preferences, and Cmd/Ctrl/Alt-; isn't a standard shortcut for anything else, so adding this shouldn't cause any conflicts or problems.
Comment 1•24 years ago
|
||
I like this idea a lot. Confirming bug
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•24 years ago
|
||
Oh, yes. Yes yes yes. (This has been in my keyboard spec since the beginning of time.) And I don't see why this shouldn't be XP, too.
Updated•24 years ago
|
Keywords: helpwanted
Comment 3•24 years ago
|
||
Since Don has left, Vishy is taking his bugs in bulk, pending reassignment. thanks, Vishy
Assignee: don → vishy
Comment 5•24 years ago
|
||
I vote for all platforms. Preferences is too hard to open quickly ;)
Hardware: Macintosh → All
Summary: Command-Semicolon should open Preferences on Mac OS → Accel-Semicolon should open Preferences (cmd-; on macos, ctrl-; on win32)
Reporter | ||
Comment 6•23 years ago
|
||
Any particular reason why this hasn't been implemented yet, after over a year? I'd do it myself if I could, but I'm not a developer and wouldn't know where to begin...
Comment 7•23 years ago
|
||
hey, aaronl, here's a[nother] bug to add to the access bug radar doc. :) i'll add it if you're busy. wasn't there some issue with this re: conflict with international keyboards...? reassign as needed...
Assignee: pchen → aaronl
Comment 8•23 years ago
|
||
The international issue is simply that the shortcut won't work on a few keyboards. However, it's still worth doing for those keyboards that can support it.
Reporter | ||
Comment 9•23 years ago
|
||
Does anybody know what existing Mac OS apps that use Cmd-; as a shortcut to Preferences do with those international keyboards? This is unfortunately not an official Apple standard, so Apple wouldn't have documentation on it. I agree if there's no semicolon and nobody knows what else to use, the shortcut can be omitted; wouldn't be any worse than it is now. The appropriate international alternatives can always be added later.
Comment 11•23 years ago
|
||
I'll do this. It bugs me.
Status: NEW → ASSIGNED
Priority: P3 → P4
Target Milestone: --- → mozilla1.1
Comment 12•23 years ago
|
||
cc'ing myself.
Comment 13•23 years ago
|
||
*** Bug 110719 has been marked as a duplicate of this bug. ***
Comment 14•23 years ago
|
||
Is there any reason why this is a 1.1 target? I recognize that it's not terribly major as bugs go, but it seems easy enough that 0.9.7 should be a reasonable target. There aren't any open issues, I don't think, from reading through this bug. Am I wrong?
Reporter | ||
Comment 15•23 years ago
|
||
Apparently it's a pain in the ass to track down exactly where this needs to be implemented. Notice the date I submitted the bug. A few search terms that might (or might not) prove useful: preferencesCmd.accesskey preferencesCmd.label key_preferences menu_preferences
Comment 16•22 years ago
|
||
Here's something that adds that binding to navigator windows. It seems a little ugly, but it works. I didn't see any way to get it installed more globally. This one only binds it in the Edit menu of navigator windows, but that seems the most useful. It can be extended later, I suspect. This works on UNIX, and assumedly win32. I would like to know if it works on Mac's, but can't test that right now. Can anyone else test this? If this isn't the right way to do it, let me know. I'll try something else. If it is good enough, any chance we can get it promoted to get into the 1.0 trunk?
Comment 17•22 years ago
|
||
Andreww, can you push this through?
Assignee: ben → andreww
Status: ASSIGNED → NEW
Comment 18•22 years ago
|
||
patch for composer... I talked with Marlon and he said it'd be better to have "context sensitive" keybindings so that when you are in composer and use command ; it'd open with composer already chosen.... in other words, behave exactly as if you had chosen edit-preferences from the menu... so I'll include patches for each app...
Comment 19•22 years ago
|
||
Comment on attachment 91036 [details] [diff] [review] changes for composer - context sensitive pref.... obsolete. I'll create one patch with all changes instead of lots of litle patches.
Attachment #91036 -
Attachment is obsolete: true
Comment 20•22 years ago
|
||
Will this conflict with the eventual nirvana of bug 77855?
Comment 21•22 years ago
|
||
whenever that bug is fixed, there will need to be many places in the UI where the code should be changed to "undo" the context sensitivity. So this will simply be another component that gets updated. That will only effect the behavior of this, but this still needs to get in and behave the way the rest of the "world" behaves right now.
Keywords: helpwanted
Whiteboard: [need r/sr]
Comment 23•22 years ago
|
||
The patch is great as far as it goes, but it misses the mail compose window. If you put the editor line in editorOverlay.xul rather than composerOverlay.xul, you get mail composer for free (and you don't have to define a new keyset) -- but then in the mail compose window it pops up the dialog with the editor prefs open, when it probably should be the mail compose prefs, so it's probably better to do the editor part the way you already have it, and add one more line for mail compose. If you want to check these in and do mail compose as a separate patch, you have r=akkana on this part (but let's leave the bug open until mail compose is done).
Comment 24•22 years ago
|
||
http://developer.apple.com/techpubs/macosx/Essentials/AquaHIGuidelines/AHIGUserInput/Reserved_an_Equivalents.html recommends Cmd+, rather than Cmd+;.
Reporter | ||
Comment 25•22 years ago
|
||
I hope that's a typo. I don't believe I've ever seen Cmd-, used for Preferences, but Cmd-; is quite common (although Apple seems determined not endorse it officially).
Comment 26•22 years ago
|
||
This RFE has a patch and a 'r' from Akkana (although it would be better to extend it immediately to the mail-compose window, to avoid another bug & patch). Can we check it in Mozilla 1.3a (1.2 is already out) ?
Comment 27•22 years ago
|
||
sure. let me refresh this patch and add support for mail compose and repost.
Comment 28•22 years ago
|
||
Safari (the new Apple-browser) uses Cmd+, so it's not a typo.
Reporter | ||
Comment 29•21 years ago
|
||
The latest iMovie and iPhoto also use Cmd-, so it looks like Apple is indeed going in this direction. A shame they decided to ignore the existing de facto standard when creating an official standard. iTunes still uses Cmd-Y, but I'm guessing this will change in the next release. Once Apple starts implementing it consistently, others will follow them, so I guess we should too.
Comment 30•21 years ago
|
||
changing summary per comment 24, adding polish keyword, and changing milestone.
Keywords: polish
Summary: Accel-Semicolon should open Preferences (cmd-; on macos, ctrl-; on win32) → Accel-Semicolon should open Preferences (cmd-, on macos, ctrl-; on win32)
Target Milestone: mozilla1.2alpha → mozilla1.5beta
Reporter | ||
Comment 31•21 years ago
|
||
ctrl-, or ctrl-; on win32? No precedent on that platform, so we should go with cross-platform consistency, unless ctrl-, is taken. I'd like to see what Apple uses in iTunes for Windows later this year...
Comment 32•21 years ago
|
||
iTunes 4.0.1 now use cmd-, too (comment 29)
Reporter | ||
Comment 33•20 years ago
|
||
iTunes for Windows uses Ctrl-, for Preferences. I recommend we follow Apple's lead on all platforms. Fixing summary to reflect this.
Summary: Accel-Semicolon should open Preferences (cmd-, on macos, ctrl-; on win32) → Accel-Comma should open Preferences (cmd-, on macos, ctrl-, on win32/linux)
Comment 34•20 years ago
|
||
this bug has been fixed on mac os x 10.3 (panther) using the latest nightly and 1.7 build (20040427). it is not fixed in mac os x 10.2 (jaguar).
Updated•15 years ago
|
Assignee: andreww → nobody
Status: ASSIGNED → NEW
QA Contact: bugzilla → keyboard.navigation
Comment 37•14 years ago
|
||
Note: Another RFE that would help multi-platform users is bug 579562: "Alt+T, O should open the Preferences dialog even on Linux, and Alt+E, N should open the Options dialog even on Windows".
Assignee | ||
Updated•5 years ago
|
Component: Keyboard: Navigation → User events and focus handling
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•