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

Set correct stem direction on pasting notes on a drum stave #22575

Merged
merged 2 commits into from May 14, 2024

Conversation

miiizen
Copy link
Contributor

@miiizen miiizen commented Apr 25, 2024

Resolves: #22313
Resolves: #22712
Resolves: Crash on changing a note or rest length in any voice other than 1 which results in a tied duration over a barline

This PR sets a chord's stem direction correctly based on the note which is being pasted on a drum stave

Screen.Recording.2024-04-25.at.15.12.26.mov

@zacjansheski
Copy link
Contributor

zacjansheski commented May 7, 2024

Tested on MacOS 14, Windows 11, Ubuntu 22.04.3. Approved
#22313 FIXED

#22712 is a very similar issue if you can take a look

@miiizen
Copy link
Contributor Author

miiizen commented May 10, 2024

@zacjansheski sorted - though found a crash unrelated to this PR when performing the steps in #22712 in voice 2 which remains unfixed as yet.

@miiizen
Copy link
Contributor Author

miiizen commented May 13, 2024

I've included a fix for that crash

@RomanPudashkin RomanPudashkin merged commit e1b7f75 into musescore:master May 14, 2024
11 checks passed
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Backport of musescore#22575

Plus some includes cleanup
Plus an entirely unrelated debug message fix
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Backport of musescore#22575

Plus some includes cleanup
Plus an entirely unrelated debug message fix
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Backport of musescore#22575

Plus some includes cleanup
Plus an entirely unrelated debug message fix
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 14, 2024
Backport of musescore#22575

Plus some includes cleanup
Plus an entirely unrelated debug message fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants