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 possible Scale Corruption in Midi Mode; disallow Lock and Group changes when scale is off #64

Merged
merged 1 commit into from
Mar 24, 2024

Conversation

jr0dsgarage
Copy link
Contributor

Issue

When in Midi mode, if the user sets the scale to OFF after using a different scale, but then turns lock scale ON, the scale is set to a corrupted scale instead of the chromatic scale.

This leads to confusion, because none of the lights are lit up, signifying that the user should be able to play any note on the keyboard, but there are some notes that don't trigger, because the scale has become corrupted.

STR

  • Set Scale to something besides OFF
  • Set Scale mode to OFF
  • Set Scale Lock to ON
  • Try to play notes on every button

Some notes won't trigger, despite the chromatic scale being selected

This fix does two things when the Scale is set to OFF (-1)

  1. When the Scale is set to OFF (aka chromatic scale), the current Lock and Group setting values are saved, and then both are set to OFF, as they are unneeded in the chromatic scale
  2. When the Scale is set to anything besides OFF, the previous lock and group settings are restored!

@jr0dsgarage jr0dsgarage changed the title Fix Scale Corruption; disallow Lock and Group changes when scale is off Fix possible Scale Corruption in Midi Mode; disallow Lock and Group changes when scale is off Mar 16, 2024
@okyeron
Copy link
Owner

okyeron commented Mar 24, 2024

is this still an issue after Mike's 1.13.1 changes?

@Quixotic7
Copy link
Contributor

That's still an issue, I'd recommend merging this.

@okyeron okyeron merged commit 5eb4f91 into okyeron:main Mar 24, 2024
1 check passed
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.

None yet

3 participants