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

fix the virtual keyboard causes the E and F to be triggered (#7604) #7976

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DevelopMan303
Copy link

It seems when the octave up/down shortcut is pressed along with a note, this issue appears. It also shows up on Windows.
My solution is to stop all the pressed notes after changing the octave, as key repeat will trigger the changed note. With this, it worked on on Ubuntu.
(BTW: my first pull request :-) )

@mkruselj
Copy link
Collaborator

mkruselj commented Feb 13, 2025

That is a venerable attempt, but... I am not sure I would consider turning all notes off if you only change octaves a good UX... no other virtual keyboard I know of works like that...

Also, not all reports of this issue mention using octave offset keys, so this would only be a partial solution.

@baconpaul
Copy link
Collaborator

But the fact that this fixes it tells us a lot

Namely a key is getting misinterpreted as a note and as an octave shift

I wonder if our key map is doing both and the answer is if you do an octave shift don’t do a key action and vice versa

@DevelopMan303
Copy link
Author

Thanks for your feedback. I agree, this may be a lite simplistic approach. I may find an optimization.
For me it looks there is an issue with the octave shift. Likely, the NoteOff is provided to the octave-shifted note.

@mkruselj
Copy link
Collaborator

If you press a normal note and hold it, but then shift the octave, the key you are holding is not going to be triggering the same note you're holding. That's why when you release that key, it doesn't send the correct note off.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants