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 beatloop_X_enabled COs when using loop_in/out #1267

Merged
merged 1 commit into from
May 30, 2017

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented May 26, 2017

and quantize is enabled. This is helpful for controllers that have a grid of pads mapped to fixed beatloop sizes. This can be tested on skins that have not been updated to use the new looping UI introduced by #1187.

Fixing Launchpad Bug 1159245

and quantize is enabled. This is helpful for controllers that have a
grid of pads mapped to fixed beatloop sizes.
@daschuer
Copy link
Member

Thank you, this works good now.

@daschuer daschuer merged commit 3521567 into mixxxdj:master May 30, 2017
@Be-ing Be-ing deleted the beatloop_size branch May 30, 2017 19:58
@daschuer
Copy link
Member

I have just noticed a different issue. We have changed that the loop out changes to the next beat when it is outside the loop. This works good if you are outside the loop and want to enlarge it.
It feels odd, if you want to make the loop smaller.
I think we should revert this behavior for the later case.

What do you think?

@Be-ing
Copy link
Contributor Author

Be-ing commented May 30, 2017

I think we should keep the current behavior. If holding loop_out with quantize snapped to the previous beat when seeking backward, that would re-introduce the weird situation with the loop enabled but the play position outside the loop boundaries. Also, snapping back at the half way point between beats might be unexpected.

@daschuer
Copy link
Member

If holding loop_out with quantize snapped to the previous beat, that would re-introduce the weird situation with the loop enabled but the play position outside the loop boundaries.

That is out of question. I have just stumbled about the situation to halve a two beat loops.
If you want to hit the beat and you miss it by some samples, nothing happens, because it snaps to the next beat. A solution would be to to distinguish these situation.

An other wired behavior is if you are behind a disabled loop, pressing Loop out set the loop out point to the nearest beat, in this case left and enables the loop. If you press the loop out button again, it snapas to the next beat right.

@Be-ing
Copy link
Contributor Author

Be-ing commented May 30, 2017

I have just stumbled about the situation to halve a two beat loops. If you want to hit the beat and you miss it by some samples, nothing happens, because it snaps to the next beat. A solution would be to to distinguish these situation.

I do not understand what the issue is or what situations you're suggesting could be distinguished. You can also press the down button on the beatloop_size spinbox or loop_halve in that situation.

An other wired behavior is if you are behind a disabled loop, pressing Loop out set the loop out point to the nearest beat, in this case left and enables the loop. If you press the loop out button again, it snapas to the next beat right.

Yes, this is how it has to be to prevent the play position from going outside the loop when resizing a loop. It is an exception to how quantize works in other situations (snapping to nearest beat).

@daschuer
Copy link
Member

daschuer commented May 31, 2017 via email

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