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 various issues with system objects and multirests #26965

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

mike-spa
Copy link
Contributor

@mike-spa mike-spa commented Mar 7, 2025

Resolves: #26930

@mike-spa mike-spa requested a review from oktophonie March 7, 2025 10:34
@mike-spa mike-spa force-pushed the fixMistakeInValidMMRestLogic branch from 6d8bc56 to f7b1328 Compare March 7, 2025 12:22
@DmitryArefiev
Copy link
Contributor

@mike-spa Tested on Win10. It still occurs on my side..

bandicam.2025-03-07.17-26-55-901.mp4

@DmitryArefiev
Copy link
Contributor

@mike-spa It's ready for testing?

@mike-spa
Copy link
Contributor Author

mike-spa commented Mar 9, 2025

@DmitryArefiev yes, it should be fixed now!

@DmitryArefiev
Copy link
Contributor

@mike-spa Variation of #26930 (comment) still occurs

bandicam.2025-03-10.01-10-20-693.mp4

@mike-spa mike-spa force-pushed the fixMistakeInValidMMRestLogic branch from f7a214a to d260b22 Compare March 10, 2025 09:07
@mike-spa mike-spa changed the title Fix error in validMMRest logic Fix various issues with system objects and multirests Mar 10, 2025
@mike-spa
Copy link
Contributor Author

@DmitryArefiev ready for another round

@DmitryArefiev
Copy link
Contributor

DmitryArefiev commented Mar 10, 2025

@mike-spa Sorry.. but I've found another two.. (I can log them separately)

  1. Size change+drag and then open parts
bandicam.2025-03-10.15-56-02-073.mp4
  1. just dragging and then create a new part (and toggling multimeasure rest)
bandicam.2025-03-10.15-58-19-970.mp4

@mike-spa mike-spa force-pushed the fixMistakeInValidMMRestLogic branch from d260b22 to 4618f5a Compare March 10, 2025 16:18
@mike-spa
Copy link
Contributor Author

@DmitryArefiev ready for another round!

@DmitryArefiev
Copy link
Contributor

@mike-spa All issues above look fine now, but I found some crashes (regression with master)

Crash 1

  1. Add Tempo
  2. Open part
  3. Toggle off multumeasure rests
  4. Undo all
bandicam.2025-03-11.14-06-58-633.mp4

Crash 2

  1. Add Tempo
  2. Change Tempo text size
  3. Open part
  4. Move Tempo (and leave it selected)
  5. Toggle off multumeasure rests (Ctrl+Shift+M)
bandicam.2025-03-11.14-08-40-300.mp4

@mike-spa mike-spa force-pushed the fixMistakeInValidMMRestLogic branch from 4618f5a to 282c58a Compare March 11, 2025 13:07
@mike-spa
Copy link
Contributor Author

@DmitryArefiev ready for another round!

@DmitryArefiev
Copy link
Contributor

@mike-spa Yeah! All seems to be fine now!

@DmitryArefiev
Copy link
Contributor

Tested #26930 on Win10, Mac13.7.2, LinuxUbuntu24.04.2 LTS - FIXED

@mike-spa

This comment was marked as resolved.

@oktophonie oktophonie requested a review from miiizen March 11, 2025 17:53
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