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

Measures extending past margin of page #20773

Closed
ShaiMartin opened this issue Dec 30, 2023 · 5 comments · Fixed by #21210
Closed

Measures extending past margin of page #20773

ShaiMartin opened this issue Dec 30, 2023 · 5 comments · Fixed by #21210
Assignees
Labels
engraving P1 Priority: High regression 4.x → 4.y Regression on a prior release

Comments

@ShaiMartin
Copy link

ShaiMartin commented Dec 30, 2023

Issue type

Engraving bug (incorrect score rendering)

Bug description

In two of my part scores, I have measures extending past the initial margin of the page, which I can't click on to edit (picture attached). Both of these only apply when a certain system is visible. In my piano part-score, if I make the left hand invisible, it aligns normally. In the guitar part-score, in which I have a notation and a tab part in, it aligns normally if I hide the tab part.

Steps to reproduce

A zipped MSCZ file is attached, the bug is evident in the guitar and piano part scores.
Score with bug in piano and guitar part scores.mscz.zip

MuseScore Version

OS: macOS 11.7, Arch.: x86_64, MuseScore version (64-bit): 4.2.0-233521125, revision: github-musescore-musescore-eb8d33c

Regression

OK in 4.0, broken in 4.1

Operating system

OS: macOS 11.7

Additional context

Screen Shot 2023-12-19 at 9 52 30 am
Screen Shot 2023-12-19 at 9 52 42 am

@cadiz2
Copy link

cadiz2 commented Dec 31, 2023

Initial discussion on forum: https://musescore.org/en/node/357957

@oktophonie oktophonie added P1 Priority: High regression 4.x → 4.y Regression on a prior release labels Dec 31, 2023
@mike-spa
Copy link
Contributor

Fixed by #21210. In the meantime @ShaiMartin the solution is to do as suggested by @MarcSabatella: right click any of the Layout Break marks in tha page (e.g. the one between bars 7-8), do "Select similar", and delete all of them.

@bkunda @oktophonie I think there is a case to be made here that the use of "Keep measures on the same system" should be discouraged as much as possible (not least because we want to get rid of it at some point). Maybe writing a warning in the handbook, if we don't already? (I think we do something similar for a few other things too).
image
I did put in some work last year to make it behave somewhat more sensibly, but it won't fix the basic fact that using this item inappropriately can force the spacing system to do something that it isn't design for. This causes bugs such as this (and indeed #20995) but also has a very signifcant performance costs.

This item should only really be used for keeping 2-measure-repeats together (that's the reason it was origiginally introduced afaik).

@cbjeukendrup
Copy link
Contributor

But I'd think we do need a way to move a measure to a previous system. I guess this can be achieved by reducing "stretch" for that measure and maybe some of the previous measures, but that feels like a hack, or at least a bit of a detour. And if you don't watch out that you reduce the street for all measures with the same amount, you'll get uneven spacing.
(However I agree the current "keep measures on same system" element is not ideal either.)

@mike-spa
Copy link
Contributor

But I'd think we do need a way to move a measure to a previous system.

We surely do, just with a proper solution :)

@oktophonie
Copy link
Contributor

See: #21213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engraving P1 Priority: High regression 4.x → 4.y Regression on a prior release
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

6 participants