432693 - Cmd+? should be used for cmd_openHelp

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Stefan [:stefanh]]

On mac, we display Cmd+? in the menu but map the command to cmd+Shift+/ (for en-US). This is done in suite/common/mac/platformCommunicatorOverlay.xul. However,  it seems that we don't need this anymore - it actually probably prevents the shortcut from working now. See bug 432112, where this was removed for Firefox. Unfortunately, this doesn't work yet on certain keyboard layouts. See bug 432632, for example. Note also bug 432388, which is supposed to fix some layouts according to bug 432632, comment #5.

Comment 1

[Stefan [:stefanh]]

We should of course wait and see what happens here. I'd like to be able to open Help too ;-)

Comment 2

[Stefan [:stefanh]]

Attachment: re-map the command and do some clean-up

There is no ui change here, we just re-map the command. I also cleaned up the comments. Bug 432632 is fixed now, so this works fine on my sv-SE keyboard. It should also work with your keyboard layout, since the issue in bug 432632, comment #2 should have been fixed by bug 432388...

Comment 3

[Karsten Düsterloh]

Comment on attachment 320183 [details] [diff] [review]
re-map the command and do some clean-up

In fact, on a German keyboard, / is actually Shift-7, thus Cmd-Shift-/ == Cmd-Shift-7 == Cmd-/ and as such is a rather weird combination for help. Cmd-? is actually Cmd-Shift-ß and as such works.

With your patch, Cmd-? still works and the erroneous Cmd-/ is gone.

You'd probably want a second review from someone with an English/American keyboard.

Comment 4

[jag (Peter Annema)]

Comment on attachment 320183 [details] [diff] [review]
re-map the command and do some clean-up

I guess there's no need to update /xpfe/communicator/resources/content/mac/platformCommunicatorOverlay.xul

sr=jag

Comment 5

[Stefan [:stefanh]]

Checking in suite/common/mac/platformCommunicatorOverlay.xul;
/cvsroot/mozilla/suite/common/mac/platformCommunicatorOverlay.xul,v  <--  platformCommunicatorOverlay.xul
new revision: 1.17; previous revision: 1.16
done
Checking in suite/locales/en-US/chrome/common/mac/platformCommunicatorOverlay.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/mac/platformCommunicatorOverlay.dtd,v  <--  platformCommunicatorOverlay.dtd
new revision: 1.12; previous revision: 1.11
done