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

Number of players is ignored when the first (or only) song in the song selection is a duet (always 6 players) #963

Open
bohning opened this issue Feb 9, 2025 · 1 comment

Comments

@bohning
Copy link
Collaborator

bohning commented Feb 9, 2025

Steps to reproduce:

  • Start USDX with just a duet song
  • Set number of players !=6 (e.g. 2)
  • Enter song selection

Image

What it should look like if two players are selected:

Image

@barbeque-squared
Copy link
Member

I think the problematic code is UScreenSong in the SetScroll function. Well, I'd say that whole file is problematic, but specifically what's happening here is that in the // Duet Singers part it's only setting to visible what it needs (if it's a Duet), and the only time it hides things is if the song is not a Duet (the else towards the end).

When it does the AddStatic etc calls in Create I don't know which AddStatic path in UMenu it takes exactly, but just search for Statics[StatNum].Visible := true; in there. It's probably not the worst default to make them visible.

The least bad solution in the current code would be to explicitly set .Visible := false in the Create function, after each AddStatic / AddText call.

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

No branches or pull requests

2 participants