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.

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

We steal Esc from host in CLAP #6838

Closed
mkruselj opened this issue Feb 5, 2023 · 10 comments
Closed

We steal Esc from host in CLAP #6838

mkruselj opened this issue Feb 5, 2023 · 10 comments
Labels
Bug Report Item submitted using the Bug Report template CLAP Issues related to CLAP plugin format UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. Windows Windows related issues
Milestone

Comments

@mkruselj
Copy link
Collaborator

mkruselj commented Feb 5, 2023

Try opening Surge CLAP in Reaper as a floating window, then press Esc. The expected behavior is that the floating window would close (like with any other plugin loaded in Reaper), but it doesn't.

VST3 works fine as expected.

@mkruselj mkruselj added Bug Report Item submitted using the Bug Report template CLAP Issues related to CLAP plugin format labels Feb 5, 2023
@baconpaul
Copy link
Collaborator

Ahh yes the clap keypress issues.
Sigh.

@mkruselj mkruselj added this to the Surge XT 1.2 milestone Feb 5, 2023
@mkruselj
Copy link
Collaborator Author

mkruselj commented Feb 5, 2023

This doesn't happen with u-he or TAL CLAPs, just for reference.

@baconpaul
Copy link
Collaborator

thanks
i wonder if it is something in CJE.

@mkruselj mkruselj added the UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. label Feb 7, 2023
@baconpaul
Copy link
Collaborator

Ahh joy of joys in reaper 6.73 on macOS, esc closes a clap just like it does a vst. lemme upgrade to 6.77 and check also.

your reaper version @mkruselj ?

@mkruselj
Copy link
Collaborator Author

mkruselj commented Mar 3, 2023

Happens in any Reaper version for me. Currently I'm running the latest prerelease (dev0302).

@baconpaul
Copy link
Collaborator

yup ok 6.77 macOS arm escape closes the window no problem.

@baconpaul
Copy link
Collaborator

Maybe try with an explicit return false; at this line

and see if that helps?

@baconpaul baconpaul added the Windows Windows related issues label Mar 3, 2023
@mkruselj
Copy link
Collaborator Author

mkruselj commented Mar 3, 2023

OK so... this is weird. That line seems to work but only if Reaper's option "Send all keyboard input to plugin" is DISABLED. WTF?!

Can you try the same on Mac?

EDIT: Actually it all works just fine even without that added line, if that option in Reaper is disabled. Baffling.

@baconpaul
Copy link
Collaborator

remind me where i find that option?

@mkruselj
Copy link
Collaborator Author

mkruselj commented Mar 3, 2023

OK so this is in the end not our bug. WIth that option enabled Esc doesn't work with TAL or u-he or even VST3, so it's not CLAP specific.

I guess the logic is - by default, Reaper intercepts Esc and uses that to close the plugin window. If we enable the option that is specifically named "Send ALL keyboard input to PLUGIN", that means plugin gets to handle that Esc and not the host. Hence the window doesn't close.

Oh well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Item submitted using the Bug Report template CLAP Issues related to CLAP plugin format UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. Windows Windows related issues
Projects
None yet
Development

No branches or pull requests

2 participants