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

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, enhancement)

Description

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch

Firefox is adding a keyboard shortcut (Ctrl+Shift+A) for the Add-on Manager. I think we should follow suit(e).

Comment 1

[[e-mail]]

(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 :-(

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

(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

[Robert Kaiser]

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.

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

(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.

Comment 5

[Bruno 'Aqualon' Escherl]

ehm, select Thread is Alt+Shift+A not Ctrl+Shift+A

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

(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

[Robert Kaiser]

(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

[[e-mail]]

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 ;-)

Comment 9

[Jens Hatlak (:InvisibleSmiley)]

(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.

Comment 10

[[e-mail]]

(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?

Comment 11

[Jens Hatlak (:InvisibleSmiley)]

(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

[Karsten Düsterloh]

(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. ;-)

Comment 13

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2

Comment 14

[[e-mail]]

Comment on attachment 482064 [details] [diff] [review]
patch v2

OK, I can live with this.

Comment 15

[Karsten Düsterloh]

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.

Comment 16

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2a [Checkin: comment 17]

Nit addressed, re-requesting review for the added AOM shortcut documentation.

Comment 17

[Jens Hatlak (:InvisibleSmiley)]

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

http://hg.mozilla.org/comm-central/rev/92544eb8a854