lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Doc: NR 4: Minor edits. (issue3406041)


From: tdanielsmusic
Subject: Re: Doc: NR 4: Minor edits. (issue3406041)
Date: Thu, 02 Dec 2010 08:34:52 +0000

LGTM; just a few nitpicks


http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1297
Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % this
\break works
indent; needs another c

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1298
Documentation/notation/spacing.itely:1298: c4 c  c c  |
spacing

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1316
Documentation/notation/spacing.itely:1316: c4 c  c c  |
do we need these two lines here?

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1556
Documentation/notation/spacing.itely:1556: music = { c'8 c' c' c' }
\relative

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode2227
Documentation/notation/spacing.itely:2227: primarily concern the amount
of vertical padding different staves
primarily affect the amount of vertical padding between the staves and
sysems.

http://codereview.appspot.com/3406041/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]