View -> Sort By key accelerators re-uses 'e' in en-US

RESOLVED FIXED in Thunderbird 52.0

Status

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: keeler, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 52.0

Thunderbird Tracking Flags

(thunderbird49 wontfix, thunderbird50 fixed, thunderbird_esr4550+ fixed, thunderbird51 fixed, thunderbird52 fixed)

Details

Attachments

(1 attachment)

Under View -> Sort By, both "Date" and "Correspondents" use "e" as the key accelerator in en-US.

Comment 1

3 years ago
Richard, can you take this on. Looks like "n" would be available for "Correspondents":
https://dxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/messenger/messenger.dtd#198
Assignee

Comment 2

3 years ago
No entity change because this is en-us specific.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8800603 - Flags: review?(jorgk)

Comment 3

3 years ago
Comment on attachment 8800603 [details] [diff] [review]
Bug1309634.patch

Thanks. You checked that "n" was available, right? - and that it was the best choice?
r=jorgk
Attachment #8800603 - Flags: review?(jorgk) → review+
Assignee

Comment 4

3 years ago
Yes, I checked this.
Assignee

Comment 5

3 years ago
https://hg.mozilla.org/comm-central/rev/ce4a68ad627976d3036c363a46f81312e100d02e
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
Assignee

Comment 6

3 years ago
Comment on attachment 8800603 [details] [diff] [review]
Bug1309634.patch

[Approval Request Comment]
User impact if declined: not always working accesskey
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): Low

I checked on ESR-45 and the accesskey is wrong too.
Attachment #8800603 - Flags: approval-comm-esr45?
Attachment #8800603 - Flags: approval-comm-beta?
Attachment #8800603 - Flags: approval-comm-aurora?

Updated

3 years ago
Attachment #8800603 - Flags: approval-comm-beta?
Attachment #8800603 - Flags: approval-comm-beta+
Attachment #8800603 - Flags: approval-comm-aurora?
Attachment #8800603 - Flags: approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.