398704 - DOM Inspector Pseudo Classes dialog does not have access keys

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, defect)

Description

[(no longer active)]

Attachment: Add the access keys

DOM Inspector Pseudo Classes dialog completely lacks access keys.  The attached patch adds the necessary access keys.  These access keys, like the corresponding labels, are hardcoded in the XUL file.

Comment 1

[Alex Vincent [:WeirdAl]]

I think the access keys, unlike the labels, should be localized.  Labels do represent pseudo-classes, but access keys map to the keyboard, which isn't always English.  So the comment above the patched lines may need changing.

Comment 2

[(no longer active)]

Hmmm, there are two points worth noting:

1. This would cuase the localized accesskeys to be shown in parenthesis besides the English labels, which is odd, at least.

2. I guess for anything but the most basic uses of the DOM Inspector, the user should have an English keyboard available.  For example, when she wants to add a new HTML element (to type in the tag name).

I'm not sure which path to take here...

Comment 3

[Shawn Wilsher :sdwilsh]

Comment on attachment 283698 [details] [diff] [review]
Add the access keys

Localizers can always choose to keep the English version, but I'd like to give them the option to change it (even with the problem your brought up).

Comment 4

[(no longer active)]

Attachment: Add the access keys (v2)

(In reply to comment #3)
> (From update of attachment 283698 [details] [diff] [review])
> Localizers can always choose to keep the English version, but I'd like to give
> them the option to change it (even with the problem your brought up).

OK, here's the new patch.

Comment 5

[[e-mail]]

Comment on attachment 284140 [details] [diff] [review]
Add the access keys (v2)

>+  <!ENTITY cbxStateHover.accesskey "h">
>+  <!ENTITY cbxStateActive.accesskey "a">
>+  <!ENTITY cbxStateFocus.accesskey "f">
I would have thought these need a localization note. sr=me with that fixed.

>\ No newline at end of file
Oops ;-)

Comment 6

[(no longer active)]

Attachment: Add the access keys (v3)

(In reply to comment #5)
> (From update of attachment 284140 [details] [diff] [review])
> >+  <!ENTITY cbxStateHover.accesskey "h">
> >+  <!ENTITY cbxStateActive.accesskey "a">
> >+  <!ENTITY cbxStateFocus.accesskey "f">
> I would have thought these need a localization note. sr=me with that fixed.

Done.

> >\ No newline at end of file
> Oops ;-)

The original file didn't have a newline at the end!  Now it will have.  :-)

Comment 7

[[e-mail]]

Comment on attachment 284495 [details] [diff] [review]
Add the access keys (v3)

The original file does end with a new line; your previous patch "removed" it.

Comment 8

[Shawn Wilsher :sdwilsh]

Comment on attachment 284495 [details] [diff] [review]
Add the access keys (v3)

r=sdwilsh

localization notes have a certain form to them, so you should fix that before asking for approval:
http://mxr.mozilla.org/seamonkey/search?string=LOCALIZATION+NOTE

Comment 9

[(no longer active)]

Attachment: Add the access keys (for checkin)

Addressed the issue in comment 8.  Asking for approval...

Comment 10

[Reed Loden [:reed]]

Checking in extensions/inspector/resources/content/viewers/dom/pseudoClassDialog.xul;
/cvsroot/mozilla/extensions/inspector/resources/content/viewers/dom/pseudoClassDialog.xul,v  <--  pseudoClassDialog.xul
new revision: 1.6; previous revision: 1.5
done
Checking in extensions/inspector/resources/locale/en-US/viewers/dom.dtd;
/cvsroot/mozilla/extensions/inspector/resources/locale/en-US/viewers/dom.dtd,v  <--  dom.dtd
new revision: 1.16; previous revision: 1.15
done