Some buttons triggered from a MIDI controller don't light up on screen (or send MIDI output)

Bug #1008238 reported by Sean M. Pappalardo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Medium
Unassigned

Bug Description

With MIDI buttons mapped to any of beatsync, cue_gotoandstop, cue_default, loop_in, loop_out, loop_halve and loop_double trigger the desired action when pressed but neither the GUI nor the controller buttons light up (using static XML <output> mappings.) When one of these is pressed on screen, it as well as the respective controller button light up correctly. The exceptions are play, flanger, pfl and hot cues; these all work correctly.

Tags: midi button led
description: updated
description: updated
summary: - Buttons triggered from a MIDI controller don't light up on screen (or
- send MIDI output)
+ Some buttons triggered from a MIDI controller don't light up on screen
+ (or send MIDI output)
description: updated
Changed in mixxx:
status: Confirmed → In Progress
Revision history for this message
Sean M. Pappalardo (pegasus-renegadetech) wrote :

Fixed in trunk r3251. ControlPushButton was setting itself to the MIDI value instead of just toggling in one case.

Changed in mixxx:
status: In Progress → Fix Committed
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

I'm pretty sure your fix for this bug re-broke Bug #906310 (which is the change that got rid of the toggling).

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Sorry -- I'm smoking something. This is unrelated to that change.

I'm not sure I agree this is the right way to fix this though. (I'm not sure I agree with that code as it was written anyway.. it should be set to 1 instead of v or !m_dValue).

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/6501

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.