Skip to content

Adding CUP button and CUP mode.#1010

Merged
daschuer merged 7 commits intomixxxdj:masterfrom
williamlemus:master
Sep 16, 2016
Merged

Adding CUP button and CUP mode.#1010
daschuer merged 7 commits intomixxxdj:masterfrom
williamlemus:master

Conversation

@williamlemus
Copy link
Copy Markdown
Contributor

https://bugs.launchpad.net/mixxx/+bug/1535468

The CUP mode has been added. It works the following way:

  • Press, pause at the cue point
  • release, play

It is available as a Cue mode, or as a button that can be learned.

This is a fixed version of PR #1001

Copy link
Copy Markdown
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now I have added only some style nits. The rest looks good to me.
I will test this soon.

Comment thread src/engine/cuecontrol.cpp Outdated
cueSet(v);
// Just in case.
m_bPreviewing = false;
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the Block above, you have still tabs.

Comment thread src/engine/cuecontrol.cpp Outdated
} else {
} else if (cueMode == CUE_MODE_CUP) {
cuePlay(v);
} else {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is a tab as well

@williamlemus
Copy link
Copy Markdown
Contributor Author

I removed the tabs. Let me know if you find any issues testing!

@daschuer
Copy link
Copy Markdown
Member

Works great, Thank you very much! LGTM,

For testing, I have used the CUP song:
https://www.youtube.com/watch?v=yhECTE2-Oqc
:-D

@daschuer daschuer merged commit 680e6d9 into mixxxdj:master Sep 16, 2016
@esbrandt
Copy link
Copy Markdown
Contributor

esbrandt commented Sep 24, 2016

Thanks for the feature.
May have found a bug with CUP mode when Quantization is enabled.

Stop the track and right-click on the PLAY button to place a cue point. Hit the CUE button. Playback starts on release.

Now enable Quantization, and again, stop the track and right-click on the PLAY button to place a cue point. Hit the CUE button and release. Nothing happens.

@williamlemus
Copy link
Copy Markdown
Contributor Author

How do I go about enabling Quantization? I want to try and reproduce it and see if I can fix it.

@esbrandt
Copy link
Copy Markdown
Contributor

esbrandt commented Oct 2, 2016

How do I go about enabling Quantization?

Click the Magnet icon in the decks button grid to enable quantization on a deck, see http://mixxx.org/manual/latest/chapters/user_interface.html#deck-options-button-grid

esbrandt added a commit to esbrandt/mixxx that referenced this pull request Oct 3, 2016
esbrandt added a commit to mixxxdj/manual that referenced this pull request Dec 5, 2016
esbrandt added a commit to mixxxdj/manual that referenced this pull request Dec 5, 2016
esbrandt added a commit to esbrandt/manual that referenced this pull request Jun 18, 2017
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.

5 participants