Skip to content

Fix Instrument change text should show full instrument name #32510

Closed
CubikingChill wants to merge 2 commits intomusescore:masterfrom
CubikingChill:my-new-branch
Closed

Fix Instrument change text should show full instrument name #32510
CubikingChill wants to merge 2 commits intomusescore:masterfrom
CubikingChill:my-new-branch

Conversation

@CubikingChill
Copy link
Copy Markdown
Contributor

@CubikingChill CubikingChill commented Mar 5, 2026

Resolves: #32588 #23018

  • I signed the CLA
  • The title of the PR describes the problem it addresses
  • Each commit's message describes its purpose and effects, and references the issue it resolves
  • If changes are extensive, there is a sequence of easily reviewable commits
  • The code in the PR follows the coding rules
  • There are no unnecessary changes
  • The code compiles and runs on my machine, preferably after each commit individually
  • I created a unit test or vtest to verify the changes I made (if applicable)

@CubikingChill CubikingChill reopened this Mar 5, 2026
@CubikingChill CubikingChill marked this pull request as draft March 5, 2026 20:10
@CubikingChill
Copy link
Copy Markdown
Contributor Author

It appears that the value "TrackName" is used extensively, from mixer track name to "select instrument" page and "change instrument" in the score. Although the instrument name in these places should ideally be the same, it appears that they are not. Variant by key is specified in "select instrument" page but not the other 2. Seems we have to fix it by either specifying variant by key in all 3 fields, or simply use longName instead.

Eitherway, this has been vary confusing.

@shoogle
Copy link
Copy Markdown
Contributor

shoogle commented Mar 11, 2026

This isn't the right solution. See #32511 (comment) for an explanation.

Instead, #32588 has the real issue and solution, which is to set instrument change text based on the output of formatInstrumentTitleOnScore().

@shoogle shoogle closed this Mar 11, 2026
@CubikingChill
Copy link
Copy Markdown
Contributor Author

I will let you know when I come up with the fixed code.

@CubikingChill CubikingChill changed the title Fix multiple clarinets sharing the same name in score. Fix Instrument change text should show full instrument name Mar 24, 2026
@CubikingChill
Copy link
Copy Markdown
Contributor Author

Can we reopen this PR?

@shoogle
Copy link
Copy Markdown
Contributor

shoogle commented Mar 27, 2026

It's better to try a new solution in a fresh PR so it's not full of messages that are no longer relevant.

Also, it means we can keep this one as a record in case someone else considers trying the same solution in the future.

If you create a new PR for this issue, you can post a link to it here for the sake of people following along.

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.

Instrument change text should show full instrument name

2 participants