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

Tango skin - Key and Rate down buttons don't respond to right-click #9163

Closed
mixxxbot opened this issue Aug 23, 2022 · 3 comments
Closed

Tango skin - Key and Rate down buttons don't respond to right-click #9163

mixxxbot opened this issue Aug 23, 2022 · 3 comments
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: beenisss
Date: 2018-03-01T17:49:58Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1752671
Tags: skin


Have tested this in the latest beta and confirmed it only affects Tango and only affects the down buttons, not the up ones.

Steps to reproduce:

  • Load the same track to two decks
  • Set them both playing and synchronise them
  • Right-click and hold on the down button for key or rate on one deck
  • Result: Nothing happens
  • Compare with doing the same for the up button
  • Result: tracks go slowly out of sync/key of selected track is adjusted upward by 10 cents
@mixxxbot
Copy link
Collaborator Author

Commented by: beenisss
Date: 2018-03-01T18:49:31Z


In res/skins/Tango/rate_pitch_key.xml, lines 32 and 140 are:
<Template src="skin:button_1state.xml">

I think they should actually be:
<Template src="skin:button_1state_right.xml">

Because otherwise the control only responds to left-click. I've updated and rebuilt my local version and that appears to have fixed it.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-03-03T14:56:00Z


Thanks for reporting this.

Right now I'm fixing & polishing some stuff in Tango #1531
Since you didn't open a PR yet, I will add those changes there.

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.1.0 milestone Aug 24, 2022
@ronso0 ronso0 added skins and removed skin labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants