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

Fix two menu reset problems #277

Merged
merged 1 commit into from
Jan 13, 2019

Conversation

baconpaul
Copy link
Collaborator

As reported in #268, the menu reset i put was too heavy handed.
Make it so it cant reset "Timbre and the like".

As reported in #261, if you are in auto-name mode and remove
your first mapping you shoudl change the name. Comment this since
the logic required more than 10 seconds thought.

@esaruoho
Copy link
Collaborator

probably codereview required, but tester's side is lgtm

@baconpaul
Copy link
Collaborator Author

@jsakkine if you are online could you just eyeball the code. I see one wonky () spacing but other than that I think it is OK. (I'll fix that wonky paren and re-push but you can look still).

As reported in surge-synthesizer#268, the menu reset i put was too heavy handed.
Make it so it cant reset "Timbre and the like".

As reported in surge-synthesizer#261, if you are in auto-name mode and remove
your first mapping you shoudl change the name. Comment this since
the logic required more than 10 seconds thought.
@jarkkojs
Copy link
Collaborator

Looking into this. Hold on.

@jarkkojs
Copy link
Collaborator

Simple question: what is EID?

@baconpaul
Copy link
Collaborator Author

Eid is the counter menus use as they build in current code. Element id perhaps? I just kept the name in the tracking variable

@baconpaul baconpaul merged commit 3aac6c2 into surge-synthesizer:master Jan 13, 2019
@jarkkojs
Copy link
Collaborator

Already merged, but looked good to me (except spacing and maybe static_cast<> instead of C style cast).

@baconpaul
Copy link
Collaborator Author

I thought I fixed all the spacing! Thanks for looking tho!

Yeah static cast in new code is better. I was sloppy copying the old cast here.

@baconpaul baconpaul deleted the menus-261-268 branch January 14, 2019 13:41
baconpaul added a commit to baconpaul/surge that referenced this pull request Jul 10, 2019
Fix two menu reset problems. Closes surge-synthesizer#261. Closes surge-synthesizer#268.

Former-commit-id: 68ac24f7583c18164982ff7357904e5d136948ff [formerly 3aac6c2]
Former-commit-id: 0816d9c67c8a90098152c173c8b772009d4f34b0
Former-commit-id: 23290d2907294bec6188493b4e2f5fcd32f8814c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants