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

[MU3] Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account. #7529

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

njvdberg
Copy link
Contributor

Solves the ignored spacer between last staff and bottom of the page.
Also it was found some combination were not handled correctly. To solve this distributeStaves() now implements it own code for finding the correct Spacer for every spacing.

A dedicated PR for master will be created shortly.

  • I signed CLA
  • I made sure the code in the PR follows the coding rules
  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code
  • I made sure the title of the PR reflects the core meaning of the issue you are solving
  • I made sure the commit message(s) contain a description and answer the question "Why do those changes fix that particular issue?" or "Why are those changes really necessary as improvements?"
  • I made sure the commit message title starts with "fix #424242:" if there is a related issue
  • I created the test (mtest, vtest, script test) to verify the changes I made

njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 19, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
@njvdberg njvdberg mentioned this pull request Feb 19, 2021
8 tasks
njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 19, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 19, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 19, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 23, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
njvdberg added a commit to njvdberg/MuseScore that referenced this pull request Feb 23, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
igorkorsukov pushed a commit that referenced this pull request Feb 25, 2021
Reimplement spacers in vertical staves adjustment to make sure all spacers are taken into account.
@vpereverzev vpereverzev closed this Mar 4, 2021
@Jojo-Schmitz
Copy link
Contributor

Still needed for 3.x though

@vpereverzev vpereverzev reopened this Mar 4, 2021
libmscore/layout.cpp Outdated Show resolved Hide resolved
@njvdberg njvdberg force-pushed the max-system-distance-3x branch 2 times, most recently from 7f33d6b to 23508f2 Compare March 11, 2021 13:20
@vpereverzev vpereverzev merged commit 5c548c3 into musescore:3.x Mar 11, 2021
@njvdberg njvdberg deleted the max-system-distance-3x branch May 15, 2021 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants