Skip to content

Changes to play count custom text behaviour #29232

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

Merged
merged 2 commits into from
Aug 21, 2025

Conversation

miiizen
Copy link
Contributor

@miiizen miiizen commented Aug 13, 2025

Resolves: #29227

This addresses the issues identified in this comment: #29227 (comment)

In the case of the styles menu being opened during text editing and the preset changed, upon ending editing the play count text auto/custom/off setting will be changed to custom.

@miiizen miiizen requested review from avvvvve, bkunda and abariska August 13, 2025 15:27
@avvvvve
Copy link

avvvvve commented Aug 13, 2025

Very nice!

@its-not-nice its-not-nice requested a review from mike-spa August 14, 2025 04:38
@abariska

This comment was marked as outdated.

@avvvvve
Copy link

avvvvve commented Aug 14, 2025

@abariska thanks for testing!

I think it would be good to fix the second item, "The text element is not focused after being unhidden" in this PR if it's a quick job.

@miiizen if the other two things are really quick 5-min fixes, feel free to address them here, otherwise I'd say they should not hold up merging this.

Copy link
Contributor

@mike-spa mike-spa left a comment

Choose a reason for hiding this comment

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

Looks good to me, but unit tests need some love

Copy link

@bkunda bkunda left a comment

Choose a reason for hiding this comment

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

Previous comments I raised seem to have been addressed here. Nice work @miiizen!

Copy link

@abariska abariska left a comment

Choose a reason for hiding this comment

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

I reported mentioned issues separately to #29379

@miiizen miiizen force-pushed the 29227-playcount-fix branch from 4851ea9 to 9723264 Compare August 20, 2025 16:03
@miiizen miiizen merged commit f4e266e into musescore:master Aug 21, 2025
13 checks passed
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.

Repeat count text is not updated after the preset change
6 participants