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

bugfix skin tango #1471

Merged
merged 1 commit into from Jan 5, 2018
Merged

bugfix skin tango #1471

merged 1 commit into from Jan 5, 2018

Conversation

nopeppermint
Copy link
Contributor

add missing right click functionality to hotcue buttons in sampler

add missing right click functionality to hotcue buttons in sampler
@nopeppermint
Copy link
Contributor Author

@Be-ing Be-ing merged commit c5660d3 into mixxxdj:2.1 Jan 5, 2018
@Be-ing
Copy link
Contributor

Be-ing commented Jan 5, 2018

LGTM thanks

@nopeppermint nopeppermint deleted the patch-2 branch January 5, 2018 17:04
@daschuer
Copy link
Member

daschuer commented Jan 5, 2018

@nopeppermint:
I am unsure if you have already signed our contributor agreement.
https://docs.google.com/a/mixxx.org/spreadsheet/viewform?formkey=dEpYN2NkVEFnWWQzbkFfM0ZYYUZ5X2c6MQ
can you do rectify this?
I will then add your name to the contributor list in the Mixxx about box.
Is it OK to use your full name from the agreement there?

@nopeppermint
Copy link
Contributor Author

done, yes full name is ok

@ronso0
Copy link
Member

ronso0 commented Jan 6, 2018

I didn't implement right-click (remove hotCue) in samplers for a very simple reason:
once you deleted a HotCue (by accident) you have NO chance to re-set it on point with current sampler interface:

  • no large waveform to see what you do
  • no mapping/hotcues to move playposition

In which scenario "clearing Hotcue is a needed functionality also in Sampler"?
Samplers are not meant to be full decks due to aspects mentioned above

@Be-ing
Copy link
Contributor

Be-ing commented Jan 6, 2018

But how often do you accidentally right click on something?

@ronso0
Copy link
Member

ronso0 commented Jan 6, 2018

Although I use samplers rather rarely, it happened to me, that's why I removed the function.
Once it happens in a regular, stressy live mixing situation where you have to deal with trackpad or trackpoint only, it's a PITA to re-set..

@nopeppermint
Copy link
Contributor Author

I didn't implement right-click (remove hotCue) in samplers ..

ok, I though that this was missing, because deck hotcue act as this as well as all hotcues on other skins..

if there is no right-click action then the Tooltip should be changed!
as Tooltip tells you, it will delete the hotcue on right-click ..

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.

None yet

4 participants