Skip to content

Show MIDI CC number even for registered/named CCs. - #1200

Merged
mfisher31 merged 1 commit into
kushview:mainfrom
BuzzBurrowes:minor-midi-monitor-tweak-pr
Oct 16, 2026
Merged

mfisher31 merged 1 commit into
kushview:mainfrom
BuzzBurrowes:minor-midi-monitor-tweak-pr

Conversation

@BuzzBurrowes

@BuzzBurrowes BuzzBurrowes commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

JUCE's built in MidiMessage::getDescription() function does not include the CC number in the returned text if the CC number is a known/registered value. EG "Sound Control 8". This is not optimal in something like the MIDI Monitor node where you sometimes need to know the actual CC number being sent by your midi controller. This change is a small tweak to the MIDI Monitor node that assures the actual CC number is always shown even for registered CC numbers. EG...

"Controller 77 - Sound Control 8: 100 channel 1"
... vs the original code...
"Controller Sound Control 8: 100 channel 1"

JUCE's build in MidiMessage::getDescription() function does not include the CC number in the returned text if the CC number is a known/registered value. EG "Sound Control 8". This is not optimal in something like the MIDI Monitor node where you sometimes need to know the actual CC number being sent by your midi controller. This change is a small tweak to the MIDI Monitor node that assures the actual CC number is always shown even for registered CC numbers. EG...

"Controller 77 - Sound Control 8: 100 channel 1"
... vs the original code...
"Controller Sound Control 8: 100 channel 1"
@BuzzBurrowes

Copy link
Copy Markdown
Contributor Author

It's fine if you don't want to take this one. It is a minor annoyance, and I am happy to keep the change local. But maybe others have this frustration as well.

@mfisher31

Copy link
Copy Markdown
Member

Works for me. Thanks again @BuzzBurrowes !

@mfisher31
mfisher31 merged commit e166258 into kushview:main Oct 16, 2026
5 checks passed
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.

2 participants