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

Update Pattern.cpp #3325

Closed
wants to merge 2 commits into from
Closed

Update Pattern.cpp #3325

wants to merge 2 commits into from

Conversation

doyousketch2
Copy link

So BB Editor changes with time signature.

If you select 2/2 time, you get 2/2 time in your BB Editor.
If you select 3/4 time, then that's what you get.

Doesn't change the Add More button, or anything like that.
It still doubles 'em so that should be fine.

Goes along with 2636

So BB Editor changes with time signature.
Add and Subtract notes
@doyousketch2
Copy link
Author

I hate git... Realized after I posted the code, it needs to update the length of the pattern to fit on the main pattern page. I like the way it looks, but I'll need to dig into the code some more to see how that's done.

@tresf
Copy link
Member

tresf commented Feb 6, 2017

I hate git...

If you sync your branch up exactly with master and push, it auto-closes the PRs. No worries, just open a new one when ready. 👍

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.

2 participants