Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Key commands like STOP (cmd + . ) don't work in Help window when it's detached #668

Closed
miczac opened this Issue Nov 30, 2012 · 6 comments

Comments

Projects
None yet
3 participants
Contributor

miczac commented Nov 30, 2012

This is probably related to #480 :
STOPing a sound or Task etc. by pressing CMD + . works within the Help window as long as this is attached (regardless if docked or undocked).
As soon as this is detached (and therefore undocked) the STOP-command apparently doesn't get passed through. I have to switch to the Editor window (where also the STOP command becomes available in the Language menu) to kill any action (like crazy sounds as Dan described it colourful).

edit:
It looks like as if this applies for other key commands as well, for instance for cmd + B for booting the server.

(found on Mac OS X 10.6.8)

@jleben jleben closed this in 3d3fc47 Dec 1, 2012

Contributor

miczac commented Jan 15, 2013

Did this commit go into 3.6.2?

Owner

jleben commented Jan 15, 2013

It did.

Contributor

miczac commented Jan 15, 2013

Sorry, apparently doesn't work here:
evaluating small sound code in Help, cmd-period has no effect.
Have to switch to IDE editor to kill the sound.
I'm on Mac OS X 10.6.8 / SC 3.6.2

Owner

jleben commented Feb 16, 2013

Confirmed. Reopening.

@jleben jleben reopened this Feb 16, 2013

Owner

jleben commented Apr 19, 2013

Fixed.

The issue was that Cmd+Period is treated specially by Mac OS, and never reaches QApplication as a QEvent. The only way it can be used to trigger a Qt shortcut is if it is a shortcut of an action in the currently active global menu for the application. However, we only had the menu with the related action active for the main window. The solution therefore was to use a common global menu bar for all windows (including detached docklets).

@jleben jleben closed this Apr 19, 2013

@sofakid sofakid pushed a commit to sofakid/supercollider that referenced this issue Apr 6, 2015

@jleben jleben scide: let cmd-period have an application-wide shortcut context
Fixes #668.
b499094

Ctrl+. doesn't work in scide.
Supercollider 3.7.1 built from HEAD d3a87c1, ubuntu 14.04.4 LTS
QT 5.2.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment