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

CMD (mac) / CTRL (windows) will revert StepSeq LFO to Sine LFO when clicking on StepSeq display with CMD/CTRL down #376

Closed
sense-amr opened this issue Jan 22, 2019 · 5 comments
Labels
Modulation Modulation related issues UI Issues related to UI look&feel
Milestone

Comments

@sense-amr
Copy link
Contributor

Pressing inside the selected LFO shape display whilst holding control results in an immediate snap back to Sine Wave LFO shape..

Control LFO click reset to Sine.zip

@sense-amr
Copy link
Contributor Author

sense-amr commented Jan 22, 2019

it appears that this action is linked to the overall action of "control click" restting to 0 value .. i would assume in this case .. 0 value = Sine wave setting on the LFO wave selection area..

I would suggest this, whilst useful on a slider.. to reset its value to 0, is not needed at all on an LFO selection.. and could easily cause confusion and misunderstanding.. again.. an example of something within the code, i presume.. that works well in some areas.. but isnt needeed at all and isnt useful in others..

@baconpaul baconpaul added bug UI Issues related to UI look&feel labels Jan 26, 2019
@baconpaul
Copy link
Collaborator

@sense-amr the behavior I see at head is

1: shift and control double click returns you to sine
2: alt and single click returns you to sine

when clicking on the graphic of the picture

both weird but want to make sure you see what I do.

The control double click matches what happens on sliders too. But on sliders a double click without any modifier held also returns you to zero. And alt click returns you to zero also.

@esaruoho esaruoho added the Modulation Modulation related issues label Feb 2, 2019
@baconpaul baconpaul added this to the 1.6.0 milestone Feb 24, 2019
@esaruoho esaruoho changed the title LFO Control key snap back to sine bug CMD (mac) / CTRL (windows) will revert StepSeq LFO to Sine LFO when clicking on StepSeq display with CMD/CTRL down Mar 5, 2019
@esaruoho
Copy link
Collaborator

esaruoho commented Mar 5, 2019

from my end on RenoiseVST2, it works like this:
Be in StepSequencer
CMD-Click on a step -> Sine
ALT-doubleclick on a step -> Sine
CTRL-doubleclick on a step -> Sine
Shift-doubleclick on a step -> Sine

so slightly different behaviour from windows. anyway. clarified the title a bit

@baconpaul
Copy link
Collaborator

@sense-amr your diagnosis is absolutely right. This is the "set default value on double click or control click" code path doing the wrong thing on this element. The code in question is the "else { p->set_value_f01(p->get_default_value" around like 1917 of SurgeGUIEditor.cpp.

baconpaul added a commit to baconpaul/surge that referenced this issue Mar 11, 2019
The LFO display (the wav) handles double click and ctrl click
like any other control - reset to default value. That's really
bad in the LFO display though, since if you ccidentally control
click or double click on your step sequencer you go back to sin.
So supress this gesture for ct_lfoshape.

Closes surge-synthesizer#376
@baconpaul
Copy link
Collaborator

Anyway I have a fix. Will sweep it in when the auto builds are done and close this issue.

baconpaul added a commit that referenced this issue Mar 11, 2019
The LFO display (the wav) handles double click and ctrl click
like any other control - reset to default value. That's really
bad in the LFO display though, since if you ccidentally control
click or double click on your step sequencer you go back to sin.
So supress this gesture for ct_lfoshape.

Closes #376
baconpaul added a commit to baconpaul/surge that referenced this issue Jul 10, 2019
The LFO display (the wav) handles double click and ctrl click
like any other control - reset to default value. That's really
bad in the LFO display though, since if you ccidentally control
click or double click on your step sequencer you go back to sin.
So supress this gesture for ct_lfoshape.

Closes surge-synthesizer#376

Former-commit-id: 82b39534a0094893baa7fd834f929d84927c9029 [formerly d7cdf68]
Former-commit-id: cbfc59ca0b546c93efb6110dacd6dff4487e8d2d
Former-commit-id: de683be145562b4cb2c51122c34a89243c306995
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Modulation Modulation related issues UI Issues related to UI look&feel
Projects
None yet
Development

No branches or pull requests

3 participants