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

Fix GH#19371: Set text line's text color #19393

Merged
merged 1 commit into from
Sep 18, 2023

Conversation

Jojo-Schmitz
Copy link
Contributor

@Jojo-Schmitz Jojo-Schmitz commented Sep 13, 2023

Resolves: #19371

IMHO better than #19391, as it also changes the texts on a change of the line color directly and not only after save/close/reopen

Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 13, 2023
@Jojo-Schmitz Jojo-Schmitz changed the title Set text line's text color Fix GH#19371: Set text line's text color Sep 13, 2023
@Jojo-Schmitz Jojo-Schmitz force-pushed the symbol-colour branch 4 times, most recently from c8da0ae to 619fb65 Compare September 13, 2023 20:40
@Jojo-Schmitz
Copy link
Contributor Author

Jojo-Schmitz commented Sep 14, 2023

Now the vtests here "fail" in the same way as in #19354:
line-colour-1 diff

Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 14, 2023
@oktophonie oktophonie added the vtests This PR produces approved changes to vtest results label Sep 14, 2023
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 14, 2023
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 14, 2023
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 14, 2023
cbjeukendrup
cbjeukendrup previously approved these changes Sep 15, 2023
@cbjeukendrup cbjeukendrup dismissed their stale review September 15, 2023 11:01

The changes need to be made in dev, not stable

@Jojo-Schmitz Jojo-Schmitz force-pushed the symbol-colour branch 2 times, most recently from 774bce5 to 393a149 Compare September 15, 2023 12:11
@cbjeukendrup cbjeukendrup merged commit a13d8b4 into musescore:master Sep 18, 2023
10 of 11 checks passed
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 18, 2023
@Jojo-Schmitz Jojo-Schmitz deleted the symbol-colour branch September 18, 2023 19:07
alexpavlov96 pushed a commit to alexpavlov96/MuseScore that referenced this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vtests This PR produces approved changes to vtest results
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lines with symbols lose colors
3 participants