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

Effect chain: (re)load & select preset after saving with a new name #4899

Merged
merged 1 commit into from
Aug 21, 2022

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Aug 16, 2022

https://bugs.launchpad.net/mixxx/+bug/1986439

Save preset with new name, and the new preset will be loaded and selected in WEffectChainMenuButton and WEffectChainPresetSelector.

Note: I didn't check wether the internediate preset unload causes sound issues. Please test.

@ronso0 ronso0 added this to the 2.4.0 milestone Aug 16, 2022
@ronso0 ronso0 force-pushed the effect-chain-select-after-save-as-new branch from 9615568 to c4563bc Compare August 20, 2022 19:05
This will select the new preset in WEffectChainMenuButton and
WEffectChainPresetSelector
@ronso0 ronso0 force-pushed the effect-chain-select-after-save-as-new branch from c4563bc to 49e301e Compare August 20, 2022 19:07
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.

Thanks. Anyone else willing to take a look?

Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

Looks and works good. More natural. Thank you.

@daschuer daschuer merged commit 593e0b1 into mixxxdj:main Aug 21, 2022
@ronso0 ronso0 deleted the effect-chain-select-after-save-as-new branch August 21, 2022 19:16
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.

3 participants