MuseScore icon indicating copy to clipboard operation
MuseScore copied to clipboard

New Courtesy Accidentals plugin

Open XiaoMigros opened this issue 1 year ago • 1 comments

The current courtesy accidentals plugin is limited in where it adds courtesy accidentals, and the UI for configuring the settings isn't particularly good-looking. In an attempt to tackle both of those issues, here's a new Courtesy Accidentals plugin.

image

  • [x] I signed the CLA
  • [x] The title of the PR describes the problem it addresses
  • [x] 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
  • [x] The code in the PR follows the coding rules
  • [x] There are no unnecessary changes
  • [x] 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)

XiaoMigros avatar Feb 03 '24 19:02 XiaoMigros

@XiaoMigros Following on your recent work on the Retrograde plugin, I was thinking it would be nice to get this one in as well. Would you be willing to rebase it?

cbjeukendrup avatar Jun 17 '25 13:06 cbjeukendrup

Can do! If possible, I'd like for the plugin to parse ornament accidentals too, meaning this would wait until #28505 is merged.

XiaoMigros avatar Jun 19 '25 15:06 XiaoMigros

Yes, let's do that!

cbjeukendrup avatar Jun 19 '25 18:06 cbjeukendrup

@XiaoMigros, a fellow user reported this issue with the Courtesy Accidentals plugin, #16565, FYI.

FrancRos31 avatar Jun 23 '25 17:06 FrancRos31

@XiaoMigros Now that https://github.com/musescore/MuseScore/pull/28505 has been merged, it would be great if you could look into this PR again!

cbjeukendrup avatar Jul 05 '25 17:07 cbjeukendrup

Will do!

XiaoMigros avatar Jul 05 '25 17:07 XiaoMigros

@XiaoMigros While working on https://github.com/musescore/MuseScore/pull/28839 I temporarily forgot about this PR, so now there will be some conflicts in configure.qml. Sorry for that!

cbjeukendrup avatar Jul 23 '25 15:07 cbjeukendrup

Sorry, I didn't mean to push that commit yet. The currently public version should now work correctly, but is not yet compatible with ornaments. I am also rewriting parts of the UI to be less clunky...

XiaoMigros avatar Nov 06 '25 13:11 XiaoMigros

Alright, no problem. Ping me when it's ready for review!

cbjeukendrup avatar Nov 06 '25 14:11 cbjeukendrup