Skip to content

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 njvdberg force-pushed the max-system-distance-3x branch from d3981f6 to 5c9cd49 Compare February 23, 2021 16:19
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
@njvdberg njvdberg force-pushed the max-system-distance-3x branch 2 times, most recently from 7f33d6b to 23508f2 Compare March 11, 2021 13:20
@njvdberg njvdberg force-pushed the max-system-distance-3x branch from 23508f2 to ef5a6fe Compare March 11, 2021 13:30
@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.

3 participants