362655 - Make "Keyboard Shortcuts" menu item pointing to online documentation

Status: RESOLVED FIXED

(Camino Graveyard :: Toolbars & Menus, defect)

Description

[froodian (Ian Leue)]

Some apps have a "Keyboard Shortcuts" menu item in the help menu (Safari, iCal, iTunes, NNW).  Since we're growing alternate and non-menu item shortcuts, I think this would be a useful addition to our menus.  The help menu currently has only three items, so I certainly don't think it would be menu overload.  We should consider doing this.

Comment 1

[Chris Lawson (gone)]

I also think this would be a very good idea.

Comment 2

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

We should discuss this at some point.  If we do want to do it for 1.1, it'll need to be done before b1.

Comment 3

[froodian (Ian Leue)]

Confirming, per pink in status meeting.

Comment 4

[froodian (Ian Leue)]

Attachment: Patch

This will need a m.o redirect to http://www.caminobrowser.org/support/docs/shortcuts/ and an addition to WebsiteDefaults.strings to have "KeyboardShortcutsPageDefault" point to it.

I also ripped out the three unused help menu items - it doesn't look like they'll be added back any time soon, and if we ever do add them back they'd be trivially easy to rewrite.

Comment 5

[froodian (Ian Leue)]

Attachment: New MainMenu.nib

Nib to go along

Comment 6

[Chris Lawson (gone)]

Comment on attachment 251828 [details] [diff] [review]
Patch

r=me on the code; didn't test or check the nib, but I trust that it's right.

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I suggest the m.o url
http://www.mozilla.org/projects/camino/shortcuts.html

We also need to clean up the cruft in WebsiteDefaults.strings when we land this.

Comment 8

[Stuart Morgan]

Comment on attachment 251828 [details] [diff] [review]
Patch

sr=smorgan for this and the appropriate strings changes.

Comment 9

[froodian (Ian Leue)]

Checked in on trunk and MOZILLA_1_8_BRANCH.  We should make sure we extra-polish that page, now that it's linked to in-app.