130318 - "Message Composition" pane accesskeys are wrong case

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P3)

Description

[Boris Zbarsky [:bzbarsky]]

UILD: 2002-03-10-21 Linux trunk nightly.

STEPS TO REPRODUCE:
1)  Open "Edit" menu
2)  Select "Preferences" option
3)  Choose the "Mail & Newsgroups" category in the list on the left
4)  Choose the "Message Composition" pane
5)  Open the "Forward Message:" dropdown
6)  Look at which letters are underlined in "Inline" and "Attachment"

ACTUAL RESULTS: the lowercase "i" and "a" are underlined.

EXPECTED RESULTS: the first (uppercase) occurrence of the letter is underlined.

According to bug 130064 accesskeys are case-sensitive and so by design.  So the
DTD just needs to be updated to use uppercase accesskeys.

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

Jean-Francois, I can take this.

Comment 2

[Jean-Francois Ducarroz]

sure, go ahead, else I will give it to varada...

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

When I fix this, I'll also fix bug 130367 at the same time.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Patch, V1.

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Patch, V2

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 73817 [details] [diff] [review]
Patch, V2

r=bzbarsky

Comment 7

[David :Bienvenu]

Comment on attachment 73817 [details] [diff] [review]
Patch, V2

sr=bienvenu

Comment 8

[Brendan Eich [:brendan]]

Comment on attachment 73817 [details] [diff] [review]
Patch, V2

a=[e-mail] for 1.0 checkin.

/be

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Fixed.

Comment 10

[Olga]

Verified on 2002-04-08-09-trunk, Win2K, Linux.

Comment 11

[Olga]

Verified on Branch 05-24-02 now.