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

RESOLVED FIXED

Status

SeaMonkey
Preferences
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Giacomo Magnini, Unassigned)

Tracking

({fixed-seamonkey1.1b})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

12 years ago
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
(Reporter)

Comment 1

12 years ago
Created attachment 242618 [details] [diff] [review]
This should be ok
Attachment #242618 - Flags: review?(neil)

Comment 2

12 years ago
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+
(Reporter)

Comment 3

12 years ago
Created attachment 242634 [details] [diff] [review]
Neil, do you like this one better?
Attachment #242634 - Flags: review?(neil)

Comment 4

12 years ago
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 5

12 years ago
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-
(Reporter)

Comment 6

12 years ago
Created attachment 243770 [details] [diff] [review]
What Ian asked.
Attachment #242618 - Attachment is obsolete: true
Attachment #242634 - Attachment is obsolete: true
Attachment #243770 - Flags: review?(iann_bugzilla)
Attachment #243770 - Flags: approval-seamonkey1.1b?

Updated

12 years ago
Attachment #243770 - Flags: review?(iann_bugzilla)
Attachment #243770 - Flags: review+
Attachment #243770 - Flags: approval-seamonkey1.1b?
Attachment #243770 - Flags: approval-seamonkey1.1b+

Updated

12 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Fixed on trunk and branch.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed-seamonkey1.1b
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.