Re: make sure that AmbitusLine is visible for small ambits (issue 4609041)

2013-09-06 Thread lemniskata . bernoullego
pushed as 1883fef6da1f91b1985ac296f0eb5bc9aee613ec https://codereview.appspot.com/4609041/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: make sure that AmbitusLine is visible for small ambits (issue 4609041)

2013-09-01 Thread lemniskata . bernoullego
Just to let you know: i've tested the patch on a bunch of simple SATB scores and the ambits look way better. https://codereview.appspot.com/4609041/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: make sure that AmbitusLine is visible for small ambits (issue 4609041)

2013-08-30 Thread lemniskata . bernoullego
https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly File input/regression/ambitus-gap.ly (right): https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly#newcode5 input/regression/ambitus-gap.ly:5: note heads are set by the @code{gap}

Re: make sure that AmbitusLine is visible for small ambits (issue 4609041)

2013-08-29 Thread k-ohara5a5a
LGTM except details https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly File input/regression/ambitus-gap.ly (right): https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly#newcode5 input/regression/ambitus-gap.ly:5: note heads are set