1238374 - Access key conflict for new IDLE checkbox in IMAP Server Settings

Status: RESOLVED FIXED

(MailNews Core :: Account Manager, defect)

Description

[rsx11m]

+++ This bug was initially created as a clone of Bug #449021 +++

Sorry for yet another follow-up bug to this (and I didn't notice it earlier).

The access key for the checkbox is 'A' which conflicts with "Account Actions" in Thunderbird and "Add Account" in SeaMonkey. Thus, depending on the focus, one or the other is chosen on Alt+A.

It appears that 'w' is still available in that pane for either application.

Comment 1

[rsx11m]

Attachment: Use 'w' instead

Note that I didn't rev useIdleNotifications.accesskey as only en-US is affected.
No problem to do so if desired.

Comment 2

[rsx11m]

Err, http://mxr.mozilla.org/comm-central/search?string=alwaysAuthenticate tells me that 'w' is also used for an NNTP-related string. However, only one or the other is not hidden in that pane, never both at the same time, thus it hopefully won't matter in practical terms.

Comment 3

[:aceman]

Comment on attachment 8706134 [details] [diff] [review]
Use 'w' instead

Review of attachment 8706134 [details] [diff] [review]:
-----------------------------------------------------------------

Yes, 'w' seems unique and does work in my tests. 'a' also works to focus the "account actions" (but the popup does not open which is already filed).

Comment 4

[rsx11m]

Thanks for the reviews, push for comm-central please.

Comment 5

[rsx11m]

Comment on attachment 8706134 [details] [diff] [review]
Use 'w' instead

[Approval Request Comment]
Regression caused by (bug #): Bug 449021
User impact if declined: confusion about shared access key
Testing completed (on c-c, etc.): works on 45.0a2 builds
Risk to taking this patch (and alternatives if risky): very low

Comment 6

[Kent James (:rkent)]

Comment on attachment 8706134 [details] [diff] [review]
Use 'w' instead

This patch has changes in localization files, which are too late for aurora. I am not convinced this issue is important enough to request late l10n.

Can this be done without changing l10n? If so, can you prepare a TB45 version of the patch?

Comment 7

[Magnus Melin [:mkmelin]]

Isn't this comparable to code changes as it doesn't just changes the l10n value, not the key?

Comment 8

[:aceman]

I think this kind of change is fine, as the localizers have already used a different key anyway. The key is only changing of en-US. So they do not need to be made aware of this change (that's why the string id does not change).

Comment 9

[Magnus Melin [:mkmelin]]

Yes. And since keys didn't change it should be upliftable

Comment 10

[Kent James (:rkent)]

Comment on attachment 8706134 [details] [diff] [review]
Use 'w' instead

OK that's fine then.

Comment 11

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/21ee57c22c62db0f2c61d15b60933ad563caf4d3
Bug 1238374 - Access key conflict for new IDLE checkbox in IMAP Server Settings. r=aceman,IanN a=IanN SM CLOSED TREE

Comment 12

[rsx11m]

Yes, there are precedence cases for en-US changes of accesskeys only.
I didn't ref the entity on purpose for that reason (thus no l01n impact).

Push for comm-aurora as well then, please.

Comment 13

[Jorg K (CEST = GMT+2)]

Aurora:
https://hg.mozilla.org/releases/comm-aurora/rev/3e2c79311fe0