Fix incorrectly defined accesskey in Customisable toolbars context menu

RESOLVED FIXED in seamonkey2.0a3

Status

--
trivial
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: wladow, Assigned: wladow)

Tracking

({polish})

Trunk
seamonkey2.0a3
polish

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 358818 [details] [diff] [review]
fix an accesskey

This accesskey definition needs to be fixed:
       label="&customizeToolbar.useDefault.label;"
       accesskey="&customizeToolbar.useDefault.label;"/>
Attachment #358818 - Flags: review?(neil)

Comment 1

10 years ago
Comment on attachment 358818 [details] [diff] [review]
fix an accesskey

Ah, so that explains why I failed to notice the abysmal choice of access key :-( Could you improve it please?
Attachment #358818 - Flags: review?(neil) → review-
(Assignee)

Comment 2

10 years ago
Created attachment 358819 [details] [diff] [review]
fix accesskeys, v2
[Checkin: Comment 4]

Right, and we can use better keys instead of 'I' and 't' as well.
Assignee: nobody → wladow
Status: NEW → ASSIGNED
Attachment #358819 - Flags: review?(neil)

Comment 3

10 years ago
Comment on attachment 358819 [details] [diff] [review]
fix accesskeys, v2
[Checkin: Comment 4]

I'm not 100% convinced with "o" but (especially in a sans serif font) it's still better than "I" :-)
Attachment #358819 - Flags: review?(neil) → review+
(Assignee)

Updated

10 years ago
Keywords: checkin-needed
Attachment #358818 - Attachment is obsolete: true
Attachment #358819 - Attachment description: fix accesskeys, v2 → fix accesskeys, v2 [Checkin: Comment 4]
Blocks: 428216
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.