-
Notifications
You must be signed in to change notification settings - Fork 404
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
Envelope Type and External Automation #206
Comments
Uh, @baconpaul, did you resolve the envelope type bug you said in this issue? I think this might have been fixed in the later versions of the synth. |
Hey while working on #702 I found the solution to this one also. |
baconpaul
added a commit
to baconpaul/surge
that referenced
this issue
Aug 22, 2019
When OSC type changes or envelope mode changes, the results are really non-local. (OSC type lays out all the sliders again; envelope mode changes the envelope control switches). To address this we can force an editor rebuild when these are toggled, kind of like we do with FX (although using a slightly different API) Closes surge-synthesizer#206 Addresses part of surge-synthesizer#702
baconpaul
added a commit
that referenced
this issue
Aug 22, 2019
When OSC type changes or envelope mode changes, the results are really non-local. (OSC type lays out all the sliders again; envelope mode changes the envelope control switches). To address this we can force an editor rebuild when these are toggled, kind of like we do with FX (although using a slightly different API) Closes #206 Addresses part of #702
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After we fixed #160 which linked DAW parameter changes with GUI changes, only envelope type was left. envelope type is different from other params in that
So while almost every parameter works, this one bug remains. I'm going to drop this here as a low priority UI bug.
If you want to figure out how to fix it here's how to diagnose
So I can get it in a state with two different bugs but not in a state with zero bugs.
The text was updated successfully, but these errors were encountered: