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

make passthrough / vinylcontrol_enabled exclusive #14237

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Jan 26, 2025

turn VC on -> passthrough off
turn passthrough on -> VC off

Passthrough stops the deck and then VC has nothing to do anyway.
And with passthrough it makes no sense to show the VC signal quality in the spinnies.
So basically this is just a small usability improvement with no harm. Or did I overlook something?

(one step to fix #14236)

when enabling one or the other, the respective other control is toggled off
Enabling Passthrough stops the deck so VC has nothing to do anyway.
Furthermore, this disables the VC signal quality display in the spinnies with Passthrough
@Holzhaus
Copy link
Member

Does the current state introduce any issues? I'm wondering because maybe some people use this behavior to use a vinyl track in their set and then continue with VC without having to re-enable it again.

I think this change, because it makes sense, but I wonder if we should remember the state before enabling pass-through and the switch VC back on automatically?

@ronso0
Copy link
Member Author

ronso0 commented Jan 27, 2025

Does the current state introduce any issues?

Nope, except the noise of VC signal renderer.
I had only GUI users in mind, and for them it's still just one click: VC on -> pass on -> VC on ...

However, if users

  • have only passthrough mapped on a controller and
  • enable VC in the GUI/with hotkey once at the start of the session

they'd now have to re-enable it again. (there are VC hotkeys Ctrl+T/Y/U/I though)

The other way around, if they have VC enable mapped they now don't have to click passthrough in the GUI anymore when switching back to VC : )

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.

VinylControl: avoid logspam when passthrough is enabled
2 participants