Closed
Bug 322242
Opened 19 years ago
Closed 16 years ago
Privacy & Security preference items missing accesskeys.
Categories
(SeaMonkey :: Security, defect)
SeaMonkey
Security
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0a2
People
(Reporter: vhaarr+bmo, Assigned: wladow)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
6.10 KB,
patch
|
Details | Diff | Splinter Review |
The following items under the Privacy & Security pref group probably need accesskeys.
Not sure if I should've filed this under the "Preferences" component or not.
Privacy & Security/
Cookies/
Cookie Manager dialog missing accesskeys.
Images/
Image Manager dialog missing accesskeys.
Popup Windows/
"Play a sound:" checkbox.
"Display an icon in the Navigator status bar" checkbox.
Allowed Web Sites dialog/
Textbox entry missing accesskey.
Forms/
Manage Sites/
Entire dialog missing accesskeys.
Passwords/
Manage Stored Passwords/
Entire dialog missing accesskeys.
Master Password/
Master Password Timeout-option boxes/
"The first time it is needed"
"Every time it is needed"
"If it has not been used for X minutes or longer"
SSL/
All checkboxes.
Certificates/
"Select Automatically" and "Ask Every Time"-option boxes.
Manage Certificates/
Entire dialog.
Manage Security Devices/
Entire dialog.
Validation/
All OCSP-optionboxes.
Manage CRLs.../
Entire dialog.
Assignee | ||
Comment 1•16 years ago
|
||
(In reply to comment #0)
> Privacy & Security/
> Cookies/
> Cookie Manager dialog missing accesskeys.
Patch follows...
> Images/
> Image Manager dialog missing accesskeys.
already fixed, patch for textbox accesskey follows
> Popup Windows/
> "Play a sound:" checkbox.
> "Display an icon in the Navigator status bar" checkbox.
already fixed
> Allowed Web Sites dialog/
> Textbox entry missing accesskey.
patch follows...
> Forms/
> Manage Sites/
> Entire dialog missing accesskeys.
no Forms pref pane in current nightlies
> Passwords/
> Manage Stored Passwords/
> Entire dialog missing accesskeys.
Password manager is being migrated to use toolkit's manager (bug 390025)
> Master Password/
> Master Password Timeout-option boxes/
> "The first time it is needed"
> "Every time it is needed"
> "If it has not been used for X minutes or longer"
already fixed
> SSL/
> All checkboxes.
already fixed
> Certificates/
> "Select Automatically" and "Ask Every Time"-option boxes.
> Manage Certificates/
> Entire dialog.
> Manage Security Devices/
> Entire dialog.
fixed
> Validation/
> All OCSP-optionboxes.
> Manage CRLs.../
> Entire dialog.
fixed
Assignee | ||
Comment 2•16 years ago
|
||
This adds required accesskeys and renames the &checkbox.label in Cookies Manager to something more descriptive.
Assignee: dveditz → wladow
Status: NEW → ASSIGNED
Attachment #338112 -
Flags: superreview?(neil)
Attachment #338112 -
Flags: review?(neil)
Updated•16 years ago
|
Attachment #338112 -
Flags: superreview?(neil)
Attachment #338112 -
Flags: superreview+
Attachment #338112 -
Flags: review?(neil)
Attachment #338112 -
Flags: review+
Comment 3•16 years ago
|
||
Comment on attachment 338112 [details] [diff] [review]
add accesskeys
> <!ENTITY button.removeallcookies.label "Remove All Cookies">
>+<!ENTITY button.removeallcookies.accesskey "A">
I think this should use the same key as "Remove All Sites" for consistency.
r+sr=me with that fixed.
Assignee | ||
Comment 4•16 years ago
|
||
Assignee | ||
Updated•16 years ago
|
Whiteboard: [checkin-needed after 2.0a1]
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Whiteboard: [checkin-needed after 2.0a1]
Comment 5•16 years ago
|
||
Comment on attachment 338129 [details] [diff] [review]
for checkin
[Checkin: Comment 5]
http://hg.mozilla.org/comm-central/rev/d441e4627247
Attachment #338129 -
Attachment description: for checkin → for checkin
[Checkin: Comment 5]
Updated•16 years ago
|
Attachment #338112 -
Attachment is obsolete: true
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.0a2
You need to log in
before you can comment on or make changes to this bug.
Description
•