Please give a pointer to the commit when marking an issue as Fixed.
This has been pushed to staging as
commit b5e8932ebc2bc86d852b5bc1a6e16e2c9c29009d
Author: Malte Meyn <[email protected]>
Date: Fri Oct 6 22:45:38 2017 +0200
Issue 5211/2: add regtest
Add a regtest for Merge_rests_engraver with \magnifyStaff.
Also add a brevis MMR to the original Merge_rests_engraver regtest.
commit 2c01a60d5a6e91e85e133efe1e2550db74aea089
Author: Malte Meyn <[email protected]>
Date: Wed Oct 4 09:58:34 2017 +0200
Issue 5211/1: fix positioning of merged rests
When used with \magnifyStaff the Merge_rests_engraver failed to position
merged rests correctly. Using 'staff-position for Rests and 'direction
for MultiMeasureRests instead of 'Y-offset fixes this.
This also fixes the positioning of brevis MMRs as in
{ \time 4/2 R\breve }
---
** [issues:#5211] Merge_rests_engraver: fix vertical rest positions**
**Status:** Fixed
**Labels:** Fixed_2_21_0
**Created:** Wed Oct 04, 2017 02:57 PM UTC by Malte Meyn
**Last Updated:** Sat Oct 14, 2017 04:41 PM UTC
**Owner:** Malte Meyn
Merge_rests_engraver: fix vertical rest positions
When used with \magnifyStaff the engraver failed to position merged
rests correctly. Using staff-position instead of Y-offset for vertical
positioning fixes this.
http://codereview.appspot.com/334740043
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
