robbielyman / seamstress

seamstress is an art engine
GNU General Public License v3.0
129 stars 12 forks source link

fix: triggers should reset instead of staying in "on" state #66

Closed p3r7 closed 1 year ago

p3r7 commented 1 year ago

before this PR, doing a params:set("my_param", 1) on a trigger param would have it stay "stuck" in this on state (like a toggle).

NEED FOR REVIEW: maybe should we do the same thing for the momentary type? idk what this type is and how it is supposed to behave.

robbielyman commented 1 year ago

my understanding is a "momentary" button is one that's on while held down, like a note on a keyboard. so i think this change is all that's needed, thank you!