Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Alt-S and alt-D access key twice in Options/Display

VERIFIED FIXED in Thunderbird 11.0

Status

Thunderbird
Preferences
--
minor
VERIFIED FIXED
8 years ago
5 years ago

People

(Reporter: nONoNonO, Assigned: aceman)

Tracking

Trunk
Thunderbird 11.0
x86
Windows XP
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
The Tools->Options tab Display now includes Fonts. Here "_S_ize" was added, but size was already there for quoted plain text messages. As a result Alt-S access key doesn't work.
(Reporter)

Comment 1

8 years ago
It's called Display Pane, tab is Formatting...
(Reporter)

Updated

8 years ago
Component: General → Preferences
QA Contact: general → preferences
Flags: wanted-thunderbird3?

Comment 2

8 years ago
confirm
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.4pre) Gecko/20091007 Shredder/3.0pre
Status: UNCONFIRMED → NEW
Ever confirmed: true
there's also two Ds
Severity: normal → minor
Summary: Als-S access key twice in Options/Display → Alt-S and alt-D access key twice in Options/Display
(Assignee)

Comment 4

6 years ago
I can do this. So what is the proposal?
Default _F_ont ? Second occurence of size -> Si_z_e ?

Also, shouldn't "Tags can be used to categorize and prioritize your messages." have an accesskey? It would focus the tag list.

Removing wanted-tb3 flag, it is outdated.
Assignee: nobody → acelists
Flags: wanted-thunderbird3?
Created attachment 574164 [details]
options display accesskey

to me,what makes the most sense is something approximate to what is shown in the screen shot
(Assignee)

Comment 6

6 years ago
Bwinton, would you accept the proposal in the screenshot?
Wayne, why do you want to change Style to _S_, instead of just leaving it as _y_, and letting the second Size keep the _S_?
Of all the options shown, I'd think the most used would be Font and Size at the top. SO thinking on it more, my original rationale for attachment 574164 [details] isn't convincing.  

How about making the second Size = _z_, keeping first Size as _S_?
(Assignee)

Comment 9

6 years ago
That is my comment 4 :)
Hmm, having f be the key for "Default font" would lead to "De_f_ault Font".  :(

Okay, so ui-r=me for keys in the following order (from top-left)
_D_efault Font, _S_ize,
Display _e_moticons as graphics,
St_y_le, Si_z_e

does that work for everyone?
(Assignee)

Comment 11

6 years ago
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #10)
> Hmm, having f be the key for "Default font" would lead to "De_f_ault Font". 
Ah, yes.

> does that work for everyone?
No problem with me, I do not use the keys. I'll patch whatever you decide.
I propose dumping a can of alphabet soup and see what floats to the top :)

seriously though, I'm cool with comment 10
(Assignee)

Comment 13

6 years ago
Created attachment 576980 [details] [diff] [review]
patch, changing accesskeys per comment 10
Attachment #576980 - Flags: ui-review?(bwinton)
Attachment #576980 - Flags: review?(dbienvenu)
Comment on attachment 576980 [details] [diff] [review]
patch, changing accesskeys per comment 10

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

WFM!
Attachment #576980 - Flags: ui-review?(bwinton) → ui-review+
(Assignee)

Comment 15

6 years ago
Can you also give review?
Status: NEW → ASSIGNED
Attachment #576980 - Flags: review?(dbienvenu) → review?(bwinton)
Comment on attachment 576980 [details] [diff] [review]
patch, changing accesskeys per comment 10

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

Okay, fine, r=me, too.

Later,
Blake.
Attachment #576980 - Flags: review?(bwinton) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/a52dab6b64f4
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
(Assignee)

Updated

6 years ago
Status: RESOLVED → VERIFIED

Updated

5 years ago
Duplicate of this bug: 521208
You need to log in before you can comment on or make changes to this bug.