-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Tuning temperaments plugin fix #26257
base: master
Are you sure you want to change the base?
Conversation
output.mp4 |
rebase needed |
modal temperaments, tuning, and modal tuning are merged into 1 plugin. Temperaments are organized by tab (western/middle eastern temperaments). Added currentTab data to history for proper "undo/redo" function. updated ui with MU components.
…tCurrentTemperament, checkCurrentTemperament and lookUpTemperament functons accordingly.
…ow be added/removed dynamically by user.
73ee4ed
to
4fd4cb2
Compare
Thank you for taking this on, @Ash-86! I have some design feedback just based on what you showed in the video. Let me know if you have any questions about this or if anything seems too tricky to implement!
All of that would come together a bit like this (sorry, not a fully polished design, just edited over a screenshot, and I didn't replace all the title case with sentence case): ![]() |
I'm also seeing a functional issue maybe to do with removing Save/Load. If you add a new temperament, then change some of the values, the values do not save after hitting "Ok" and reopening the dialog. I would expect that if I edit those values, they get saved to my custom temperament when I hit "OK". (Didn't record this, but in the video I close and reopen the plugin via the menu) Screen.Recording.2025-02-26.at.5.36.51.PM.mov |
Hi @avvvvve, I implemented most of the above and i have a couple of questions. Regarding item 6, currently, temperament values are reset when a temperament is clicked. I think that was even the original behaviour. Considering that, you still prefer to add a reset button?
RIght. I didn't implement autosave for changes when hitting 'OK' since you can just add a new temperament with the new changes and sometimes you wouldn't want to save whatever experimental tweaking you did. That said, if you think it is better, I could add a save button or have it automatically save temperament changes when hitting 'OK'. Oh, and i the popup dialog just refused to have the same width as the add button and ended up slightly wider. Any tip on that would be great :) |
I had accidentally rebsed the branch in the old pr, and it closed automatically after i tried resetting. Couldnt reopen so i opened this one instead - sorry :(