Enabling keylock on playing track resets key

[Ilkka Tuohela]

If you enable keylock on a playing track with rate fader not in center position, the key jumps to 0 value.

This sounds quite bad if you are playing the track, and besides doesn't feel like correct choice anyway: if user has set the key with rate fader, it would seem to make more sense to keep current key when locked. I propose we at least add an option to keep the current key in such case, or make it default.

One issue I see with keeping the key is what should happen if pitch slider is moved after activating keylock and keylock is disabled again: maybe the key should jump to value matching the slider position in this case?

[Owen Williams (ywwg)]

This is actually a tricky fix, even though it wouldn't be a ton of lines of code. It will involve talking to SoundTouch and setting both the tempo and pitch parameters such that the pitch is the same as the previous value, but the tempo is too. (Right now I think Mixxx just sets the tempo and leaves the pitch alone). I'm not sure what the interaction is between the two variables, so I'm not sure what to set for the right behavior.

[jus (jus)]

Currently discussed in lp:1258617

[Daniel Schürmann (daschuer)]

Now the key locks "locks" the pitch at the current pitched key. So the original bug is fixed.
I think RJ has done it, so I will assign the bug to him.

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/6582