Closed Bug 357096 Opened 14 years ago Closed 14 years ago

Preferences -> Privacy & Security -> Popup Windows misses a few accesskeys

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: prometeo.bugs, Unassigned)

Details

(Keywords: fixed-seamonkey1.1b)

Attachments

(1 file, 2 obsolete files)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; it; rv:1.8.0.7) Gecko/20060913 Fedora/1.5.0.7-1.fc5 Firefox/1.5.0.7 pango-text
Build Identifier: 

Play a sound -> P
Display an icon... -> D

While in there, use v for Preview (as in prefs->navigator->downloads and in prefs->mailnews->notifications).

Reproducible: Always
Attached patch This should be ok (obsolete) — Splinter Review
Attachment #242618 - Flags: review?(neil)
Comment on attachment 242618 [details] [diff] [review]
This should be ok

> <!ENTITY selectSound.label              "Select...">
Interestingly Mail & Newsgroups Notifications uses _B_rowse...
Attachment #242618 - Flags: review?(neil) → review+
Attachment #242634 - Flags: review?(neil)
Comment on attachment 242634 [details] [diff] [review]
Neil, do you like this one better?

> <!ENTITY popupBlock.label               "Block unrequested popup windows">
>-<!ENTITY popupBlock.accesskey           "B">
>+<!ENTITY popupBlock.accesskey           "k">
Not sure... maybe this should be w, unless IanN has any better ideas...

>+<!ENTITY selectSound.label              "Browse...">
>+<!ENTITY selectSound.accesskey          "B">
Attachment #242634 - Flags: review?(neil) → review?(iann_bugzilla)
Comment on attachment 242634 [details] [diff] [review]
Neil, do you like this one better?

>Index: suite/locales/en-US/chrome/common/pref/pref-popups.dtd
>===================================================================
> <!ENTITY popupBlock.label               "Block unrequested popup windows">
>-<!ENTITY popupBlock.accesskey           "B">
>+<!ENTITY popupBlock.accesskey           "k">
My preference is to leave this as "B"
> 
> <!ENTITY popupExceptions.label          "Allowed Sites...">
> <!ENTITY popupExceptions.accesskey      "A">
> 
> <!ENTITY whenBlock.description          "When a popup window has been blocked:">
> <!ENTITY playSound.label                "Play a sound:">
>+<!ENTITY playSound.accesskey            "P">
Possibly use "s" here and "P" later on for "Preview"
> 
>-<!ENTITY selectSound.label              "Select...">
>-<!ENTITY selectSound.accesskey          "e">
>+<!ENTITY selectSound.label              "Browse...">
>+<!ENTITY selectSound.accesskey          "B">
Possibly use "r" here

> <!ENTITY previewSound.label             "Preview">
>-<!ENTITY previewSound.accesskey         "r">
>+<!ENTITY previewSound.accesskey         "v">
Possibly use "P" here or "e" if "P" is still used for "Play a sound:"
Attachment #242634 - Flags: review?(iann_bugzilla) → review-
Attached patch What Ian asked.Splinter Review
Attachment #242618 - Attachment is obsolete: true
Attachment #242634 - Attachment is obsolete: true
Attachment #243770 - Flags: review?(iann_bugzilla)
Attachment #243770 - Flags: approval-seamonkey1.1b?
Attachment #243770 - Flags: review?(iann_bugzilla)
Attachment #243770 - Flags: review+
Attachment #243770 - Flags: approval-seamonkey1.1b?
Attachment #243770 - Flags: approval-seamonkey1.1b+
Status: UNCONFIRMED → NEW
Ever confirmed: true
Fixed on trunk and branch.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.