Closed
Bug 1324292
Opened 8 years ago
Closed 8 years ago
need accesskey for "Close" button in find and replace when composing message
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 53.0
People
(Reporter: wsmwk, Assigned: Paenglab)
Details
Attachments
(1 file)
|
4.77 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
This keeps biting me.
open compose
do ctrl+F
do alt+L
close has no accesskey. "L" is available :)
| Assignee | ||
Comment 1•8 years ago
|
||
"L" isn't a good letter for accesskeys because the lowercase l is too narrow to have a remarkable underscore line.
I choose "C" and changed the "Match exact case" to "M". If the "c" would be on _c_ase, then I would leave it, but on exa_c_t it's not very obvious.
I also haven't changed the caseSensitiveCheckbox.accesskey entity to caseSensitiveCheckbox1.accesskey because it's not likely other locales need also to change this accesskey.
Aceman, sorry for the lot of white space changes in EdReplace.xul but after this bug it's clean. ;)
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8819671 -
Flags: review?(acelists)
Why does the dialog even have a Close button? I couldn't find any other one with such a button.
Isn't ESC enough for a key?
| Assignee | ||
Comment 3•8 years ago
|
||
The Check Spelling dialog has one too. And it has also "C" as accesskey.
Comment on attachment 8819671 [details] [diff] [review]
20894.patch
Review of attachment 8819671 [details] [diff] [review]:
-----------------------------------------------------------------
Ok then.
Attachment #8819671 -
Flags: review?(acelists) → review+
| Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 5•8 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 53.0
Updated•8 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•