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.

Sign up for GitHub

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

Jump to bottom

LFO phase parameter wraparound #6568

Open
Andreya-Autumn opened this issue Aug 23, 2022 · 0 comments
Open

LFO phase parameter wraparound #6568

Andreya-Autumn opened this issue Aug 23, 2022 · 0 comments
Labels
Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) Discussion Modulation Modulation related issues
Milestone
Surge XT 2.0

Comments

@Andreya-Autumn
Copy link
Collaborator

...meaning the value at the bottom of the slider is always = that at the top. That means when modulating this slider you sometimes gotta be careful to omit the last % from the modulation range or get weird zips if the start and end value are different.

I don't know when I'd want this and I'm somewhat regularly running into scenarios where I really don't. Maybe it should go to 359 degrees instead of 360?
@Andreya-Autumn Andreya-Autumn added UI Issues related to UI look&feel Modulation Modulation related issues UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. Discussion Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) labels Aug 23, 2022
@Andreya-Autumn Andreya-Autumn added this to the Surge XT 2.0 milestone Aug 23, 2022
@mkruselj mkruselj removed UI Issues related to UI look&feel UX Issues related to user experience (UX) - mouse, touch, keyboard, MIDI inputs, etc. labels Aug 23, 2022
@mkruselj mkruselj changed the title LFO phase parameter wraps around. LFO phase parameter wraparound Aug 30, 2022
This was referenced Mar 5, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) Discussion Modulation Modulation related issues
Projects
None yet
Milestone
Surge XT 2.0
Development

No branches or pull requests
2 participants
@mkruselj @Andreya-Autumn