Closed Bug 322242 Opened 19 years ago Closed 16 years ago

Privacy & Security preference items missing accesskeys.

Categories

(SeaMonkey :: Security, defect)

defect
Not set
minor

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)

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.
(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
Attached patch add accesskeys (obsolete) — Splinter Review
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)
Attachment #338112 - Flags: superreview?(neil)
Attachment #338112 - Flags: superreview+
Attachment #338112 - Flags: review?(neil)
Attachment #338112 - Flags: review+
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.
Whiteboard: [checkin-needed after 2.0a1]
Keywords: checkin-needed
Whiteboard: [checkin-needed after 2.0a1]
Attachment #338129 - Attachment description: for checkin → for checkin [Checkin: Comment 5]
Attachment #338112 - Attachment is obsolete: true
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.

Attachment

General

Created:
Updated:
Size: