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

Numark PartyMix: fix EQ (script binding) display name #13255

Merged
merged 1 commit into from
May 22, 2024

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented May 21, 2024

Noticed while testing the mapping IO table with a built-in mapping.

Group [EqualizerRack1_[ChannelN]_Effect1] will not be translated in the mapping IO GUI, and it's not required since it's already set via components. Simply use [ChannelN] to get the translated string 'Deck N' in the GUI.

group [EqualizerRack1_[ChannelN]_Effect1] will not be translated in the mapping IO GUI,
and it's not required since it's already set via components.
Simply use [ChannelN] to get the translated string 'Deck N' in the GUI
Copy link
Member

@Swiftb0y Swiftb0y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I doubt there is much value to having this translated in a mapping thats not made to be customized through the input-mapping table, but sure.

@Swiftb0y Swiftb0y merged commit 88d7f0b into mixxxdj:2.4 May 22, 2024
14 checks passed
@ronso0
Copy link
Member Author

ronso0 commented May 22, 2024

It's not about the translation itself but about the sorting. With the previous group [EqualizerRack1_.. the items would appear at the end/top because of leading [, not near Effects... Even with the original tr string (Deck N) all deck controls will now be grouped.

@ronso0 ronso0 deleted the numark-party-mix-eq branch May 22, 2024 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants