Fix crashing of tuning plugins by Bill Hails #20537
Open
+154
−154
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves: #19326
When developing my own tuning plugin for MS4 on MacOS, I noticed that since 4.1, Qt Text components would crash only on Mac (fine on windows 10+). I then noticed that the vanilla tuning plugins by Bill Hails also crashes for the same reason.
While I made the "simplest" changes to fix this crash (updating
Qt.Controls
version in the plugin implementations), one downside of this fix is the loss of theming that was inherited by earlierQt.Controls
packages (somehow). Consider the before and after:This may raise accessibility concerns, in addition to the fact that these changes will be made to both Windows and Mac, despite the crash only occurring on Mac. My preferred patch (if I could budget the time) would be either (i) to deploy the patched version only for Mac, or (ii) to rewrite the plugin using
MuseScore.UIComponents
as mentioned here.Nonetheless, I hope that this PR can at least inspire the changes that fixes these plugins for MacOS, as they have served me well for microtonal compositions.