|
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/
[Prev in Thread] | Current Thread | [Next in Thread] |