322240 - Various pref panel items missing accesskeys.

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, defect)

Description

[Vidar Haarr (not reading bugmail)]

Various pref panel items missing accesskeys that I didn't find a specific Component for:

  Appearance/
    Themes/
      "Theme to use"-box.
      "Get New Themes"-link.
    Language and Content Packs/
      "Installed language packs:"-box.
      "Installed content packs:"-box.

  Nagivator/
    "Display on"-combobox.
    Languages/
      "Languages in order of preference:"-box.
    Helper Applications/
      "File types:"-box.
      "When encountered:"-textbox.
      PFS checkbox.
    Smart Browsing/
      Domain guessing checkbox.
      Advanced.../
        Entire dialog.

  Roaming User/
    Server Information dropdown.

Comment 1

[Serge Gautherie (:sgautherie)]

(Filter "spam" on 'prefs-nobody-20080612'.)

Comment 2

[Vlado Valastiak [:wladow]]

Attachment: add accesskeys, v1

>  Appearance/
>    Themes/
>      "Theme to use"-box.
>      "Get New Themes"-link.
>    Language and Content Packs/
>      "Installed language packs:"-box.
>      "Installed content packs:"-box.
panes removed in bug 372856 and bug 411215

>  Nagivator/
>    "Display on"-combobox.
fixed in this patch

>    Languages/
>      "Languages in order of preference:"-box.
fixed in this patch

>    Helper Applications/
>      "File types:"-box.
>      "When encountered:"-textbox.
>      PFS checkbox.
replaced by new prefpane in Bug 417590

>    Smart Browsing/
>      Domain guessing checkbox.
>      Advanced.../
>        Entire dialog.
replaced by new prefpane in bug 411226

>  Roaming User/
>    Server Information dropdown.
pane removed from 2.0 builds

Comment 3

[[e-mail]]

Comment on attachment 354059 [details] [diff] [review]
add accesskeys, v1

>-      <label control="activeLanguages">&languages.customize.active.label;</label>
>+      <label control="activeLanguages"
>+             value="&languages.customize.active.label;"
>+             accesskey="&languages.customize.active.accesskey;"/>
Why change the content to the value attribute? (There's a slight difference as the attribute version won't wrap, which might be a problem in some languages?)

>+<!ENTITY navRadio.label                 "Display on">
>+<!ENTITY navRadio.accesskey             "a">
I think "n" would make a better access key.

Comment 4

[Vlado Valastiak [:wladow]]

Attachment: add accesskeys, v2 [Checkin: Comment 6]

- fixed label value back to content
- changed navRadio.accesskey to "n", although I personally don't like acceskeys being placed under the last char in a string

Comment 5

[[e-mail]]

Comment on attachment 355012 [details] [diff] [review]
add accesskeys, v2
[Checkin: Comment 6]

>I personally don't like access keys being placed under the last char in a string
Yeah, but neither are you supposed to use vowels.

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 355012 [details] [diff] [review]
add accesskeys, v2
[Checkin: Comment 6]

http://hg.mozilla.org/comm-central/rev/f6df39970eca