Bryan Assignee
	Description • 19 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b3) Gecko/20050713 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b3) Gecko/20050713 Firefox/1.0+ ID:2005071315

Tools-> Options-> Cache-> Clear Cache Now and Help buttons have the same
keyboard shortcut _h_ and thus one doesn't work when using the keyboard shortcut.

Reproducible: Always

Steps to Reproduce:
1. Download latest Deer Park build
2. Launch DP :-)
3. Click Tools-> Options-> Cache->
4. Observe that both the Clear Cache Now and Help buttons have the same keyboard
shortcut _h_ and thus one doesn't work when using the keyboard shortcut.

Actual Results:  
Observe that both the Clear Cache Now and Help buttons have the same keyboard
shortcut _h_ and thus one doesn't work when using the keyboard shortcut.

Expected Results:  
The Clear Cache Now and Help buttons should have DIFFERENT keyboard shortcuts
and  that clicking ALT+Keyboard Shortcut would actually work!

Any of the letters in the word "Clear" (Clear Cache Now) are available for use
as a keyboard shortcut in that dialog.

	Steve England [:stevee]
	Comment 1 • 19 years ago

*** Bug 300698 has been marked as a duplicate of this bug. ***

	u60234
	Comment 2 • 19 years ago

Actually, the only key available for "Clear Cache Now" is "N". Due to bug
143065, all tab sheets in the Privacy panel must share the same set of accesskeys.

Because of this, the accesskey for "Remember visited pages for the last x days"
is also broken. "R" is taken by "Remove Master Password" on the Password tab.
The "P" for "Remember Password" is also used for "use up to:" on the Cache tab.

	u60234
	Comment 3 • 19 years ago

Use "b" for "Remember Password", "t" for "Use up to:", and "N" for "Clear Cache
Now".

	Bryan Assignee
	Comment 4 • 19 years ago

EEK!!!  A few ppl working on this I see.  I fixed about ten or so accesskey
issues in Tools-> Options last night and forwarded the changes to Gavin.  I
believe he's going to create the diff's tonight and the subsequent patch.

	u60234
	Comment 5 • 19 years ago

Comment on attachment 189310 [details] [diff] [review]
patch

Oh, feel free then to take this bug if a more comprehensive patch is
forthcoming.

	Bryan Assignee
	Comment 6 • 19 years ago

This should be better then what we have now and resolves the bustage from the
landing of the bug to add "Help" buttons to the Options dialogs.

	Bryan Assignee
	Comment 7 • 19 years ago

I've got an even more comprehensive patch coming up in the next day or two! 
I've added about 40 additional accesskeys and fixed those that were "broken".

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 8 • 19 years ago

This has really turned in to bug 191642, so duping. Bryan, post the new patch in
that bug, please.

*** This bug has been marked as a duplicate of 191642 ***