The default bug view has changed. See this FAQ.

Port |Bug 465090 - Add keyboard shortcut to open Addons Manager| to SeaMonkey

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
UI Design
--
enhancement
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1b2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
Created attachment 470041 [details] [diff] [review]
patch

Firefox is adding a keyboard shortcut (Ctrl+Shift+A) for the Add-on Manager. I think we should follow suit(e).
Attachment #470041 - Flags: superreview?(neil)
Attachment #470041 - Flags: review?(neil)

Comment 1

7 years ago
(In reply to comment #0)
> Firefox is adding a keyboard shortcut (Ctrl+Shift+A) for the Add-on Manager. I
> think we should follow suit(e).
That's Select Thread in MailNews :-(
(Assignee)

Comment 2

7 years ago
(In reply to comment #1)
> That's Select Thread in MailNews :-(

That's really unfortunate. Same with TB. Maybe we should see what they'll do?

I wonder whether we can remove that shortcut over there--or will people kill us (esp. me) for that? I didn't know this existed at all! Edit/Select/Flagged Messages hat no shortcut either BTW.

Or maybe add the new shortcut only to browser windows? I mean, now that it's a tab...

Comment 3

7 years ago
Good luck with finding a free shortcut in mailnews. I'm not aware of any, I know we already had large problems finding one for download manager.
(Assignee)

Comment 4

7 years ago
(In reply to comment #3)
> Good luck with finding a free shortcut in mailnews. I'm not aware of any, I
> know we already had large problems finding one for download manager.

If you had read my comment 2 you would have seen that I have no intention to find a shortcut that is currently free in MailNews. I'm well aware of the situation.
ehm, select Thread is Alt+Shift+A not Ctrl+Shift+A
(Assignee)

Comment 6

7 years ago
(In reply to comment #5)
> ehm, select Thread is Alt+Shift+A not Ctrl+Shift+A

Only on Linux/Unix (see below). But thanks for noticing, this might be our way out (using that shortcut on all platforms)! :-) Neil, KaiRo, what do you think?

<http://mxr.mozilla.org/comm-central/search?string=key_selectThread>

Comment 7

7 years ago
(In reply to comment #2)
> Or maybe add the new shortcut only to browser windows? I mean, now that it's a
> tab...

No, even if it opens in a tab, it's a cross-component feature. And that's it's currently a browser tab is a technical detail that will not survive indefinitely anyhow.

(In reply to comment #6)
> Neil, KaiRo, what do you think?

I think that this all is a really bug mess and we have too many options with shortcuts, and MailNews is too complicated. But that leads too far.

I'll let Neil decide, he knows more than any of us about shortcuts.

Comment 8

7 years ago
The only idea I could come up with was to move select thread to Shift+A.

But it's Mnyromyr's module, so he gets final say ;-)
(Assignee)

Comment 9

7 years ago
(In reply to comment #8)
> The only idea I could come up with was to move select thread to Shift+A.

That's Archive. ;-)

> But it's Mnyromyr's module, so he gets final say ;-)

So, here some thoughts why I think making Select Thread Alt+Shift+A on all platforms and making Ctrl+A open the Add-on Manager is the way to go:
1. Ctrl+Key shortcuts are first-class. People who use add-ons open the Add-on Manager quite frequently, from whichever module they're currently in (say, browser or MailNews).
2. Alt+Shift shortcuts are not first-class. As is Select Thread (contrary to Select).
3. Alt+Shift shortcuts are semi-reserved for website shortcuts (accesskey HTML attribute) which are more likely to appear in content opened in the browser (any website) than MailNews (feeds only)
4. Alt+Shift+A is already used on Unix/Linux, so achieving consistency there would also mean less confusion. 
5. Firefox uses Ctrl+A for the Add-on Manager, so this would be consistent (I put this point last on purpose).

Please ignore the current patch for now and consider the above only.
(In reply to comment #9)
> (In reply to comment #8)
> > The only idea I could come up with was to move select thread to Shift+A.
> That's Archive. ;-)
Sigh. I guess "A" was deemed too dangerous.

> So, here some thoughts why I think making Select Thread Alt+Shift+A on all
> platforms and making Ctrl+A open the Add-on Manager is the way to go:
You mean Ctrl+Shift+A, right?
(Assignee)

Comment 11

7 years ago
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > So, here some thoughts why I think making Select Thread Alt+Shift+A on all
> > platforms and making Ctrl+A open the Add-on Manager is the way to go:
> You mean Ctrl+Shift+A, right?

Good catch. I have absolutely no intention to re-purpose Ctrl+A. ;-)

Comment 12

7 years ago
(In reply to comment #11)
> > You mean Ctrl+Shift+A, right?
> Good catch. I have absolutely no intention to re-purpose Ctrl+A. ;-)

Phewy, you really made me worry. ;-)
(Assignee)

Comment 13

7 years ago
Created attachment 482064 [details] [diff] [review]
patch v2
Attachment #470041 - Attachment is obsolete: true
Attachment #482064 - Flags: superreview?(neil)
Attachment #482064 - Flags: review?(mnyromyr)
Attachment #470041 - Flags: superreview?(neil)
Attachment #470041 - Flags: review?(neil)
Comment on attachment 482064 [details] [diff] [review]
patch v2

OK, I can live with this.
Attachment #482064 - Flags: superreview?(neil) → superreview+

Comment 15

7 years ago
Comment on attachment 482064 [details] [diff] [review]
patch v2

>diff --git a/suite/locales/en-US/chrome/common/tasksOverlay.dtd b/suite/locales/en-US/chrome/common/tasksOverlay.dtd
> <!ENTITY downloadManagerCmd.label "Download Manager">
> <!ENTITY downloadManagerCmd.accesskey "D">
> <!ENTITY downloadManagerCmd.commandkey "j">
> 
> <!ENTITY addonsManagerCmd.label "Add-on Manager">
> <!ENTITY addonsManagerCmd.accesskey "A">
>+<!ENTITY addonsManagerCmd.commandkey "A">

This should be a lower case "a".

r=me with that.
(Assignee)

Comment 16

7 years ago
Created attachment 483854 [details] [diff] [review]
patch v2a [Checkin: comment 17]

Nit addressed, re-requesting review for the added AOM shortcut documentation.
Attachment #482064 - Attachment is obsolete: true
Attachment #483854 - Flags: superreview+
Attachment #483854 - Flags: review?(mnyromyr)
Attachment #482064 - Flags: review?(mnyromyr)

Updated

7 years ago
Attachment #483854 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 17

7 years ago
Comment on attachment 483854 [details] [diff] [review]
patch v2a [Checkin: comment 17]

http://hg.mozilla.org/comm-central/rev/92544eb8a854
Attachment #483854 - Attachment description: patch v2a → patch v2a [Checkin: comment 17]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
Depends on: 606208
You need to log in before you can comment on or make changes to this bug.