Skip to content
New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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
Closed

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

miczac opened this issue Nov 30, 2012 · 6 comments
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs. env: SCIDE os: macOS
Milestone
3.6.4

Comments

@miczac
Copy link
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 as completed in 3d3fc47 Dec 1, 2012
jleben added a commit that referenced this issue Dec 1, 2012
@jleben
scide: let cmd-period have an application-wide shortcut context
05cc476 
Fixes #668.
@miczac
Copy link
Contributor Author

miczac commented Jan 15, 2013

Did this commit go into 3.6.2?

@jleben
Copy link
Member

jleben commented Jan 15, 2013

It did.

@miczac
Copy link
Contributor Author

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

@jleben
Copy link
Member

jleben commented Feb 16, 2013

Confirmed. Reopening.

@jleben jleben reopened this Feb 16, 2013
@jleben
Copy link
Member

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 as completed Apr 19, 2013
sofakid pushed a commit to sofakid/supercollider that referenced this issue Apr 6, 2015
@jleben
scide: let cmd-period have an application-wide shortcut context
b499094 
Fixes supercollider#668.
@albreche
Copy link

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
Assignees
No one assigned
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs. env: SCIDE os: macOS
Projects
None yet
Milestone
3.6.4
Development

No branches or pull requests
3 participants
@jleben @miczac @albreche