Late Night Skin - Changing musical key by clicking in interface

Bug #1475367 reported by Raf Van De Meirssche
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
jus

Bug Description

This is ONLY in the LateNight skin.
Deere and Shade function as I suppose they should.

Build 1.12 r5481.
Both Ubuntu 14.04 LTS 64-bit and Windows 7 Home Premium 64-bit.

When I press one of the little triangular arrows next to the KEY button, I see no change.
I have to press the button 4 times before I see the key notation change.

I am not a musician and do not have that musical ear, but I think the musical key doesn't change either before that 4th press.
This is for your information, because I suppose that changing by a quarter of a tone has little sense.
But then again ... I'm not a musician.

On the other hand ... Deere and Shade do change on first instance as I would expect it to do.
(One was a knob, don't remember which)

Tags: easy polish skin
Revision history for this message
jus (jus) wrote :

This is because the skin uses the ``pitch_down_small`` & ``pitch_up_small`` controls for the buttons, which changes the key only by a tiny amount. The following allows audible changes on click, and small changes on right-click:

https://github.com/mixxxdj/mixxx/blob/master/res/skins/LateNight/deck.xml#L35 ff.
and
https://github.com/mixxxdj/mixxx/blob/master/res/skins/LateNight/deck.xml#L79 ff.
replace with

<Connection>
  <ConfigKey persist="true">[Channel<Variable name="channum"/>],pitch_down</ConfigKey>
  <ButtonState>LeftButton</ButtonState>
</Connection>
<Connection>
  <ConfigKey persist="true">[Channel<Variable name="channum"/>],pitch_down_small</ConfigKey>
  <ButtonState>RightButton</ButtonState>
</Connection>

tags: added: easy polish skin
Changed in mixxx:
status: New → Confirmed
Revision history for this message
jus (jus) wrote :

geez, you still cant edit comments on lauchpad.

Of course you need a different pair of controls for https://github.com/mixxxdj/mixxx/blob/master/res/skins/LateNight/deck.xml#L79 ff.
<Connection>
  <ConfigKey persist="true">[Channel<Variable name="channum"/>],pitch_up</ConfigKey>
  <ButtonState>LeftButton</ButtonState>
</Connection>
<Connection>
  <ConfigKey persist="true">[Channel<Variable name="channum"/>],pitch_up_small</ConfigKey>
  <ButtonState>RightButton</ButtonState>
</Connection>

Revision history for this message
jus (jus) wrote :
Changed in mixxx:
milestone: none → 2.0.0
status: Confirmed → Fix Committed
importance: Undecided → Low
assignee: nobody → jus (jus)
RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/8157

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.