lilypond-devel
[Top][All Lists]
Advanced

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

Re: Doc: NR add predefined paper sizes to appendix (issue 5783075)


From: tdanielsmusic
Subject: Re: Doc: NR add predefined paper sizes to appendix (issue 5783075)
Date: Sun, 18 Mar 2012 17:54:02 +0000

LGTM, James, apart from a couple of nitpicks (but I
didn't check the sizes :)

Might be worth raising an issue about the lack of an
automated appendix once this is pushed.

Thanks

Trevor



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

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacing.itely#newcode211
Documentation/notation/spacing.itely:211: @code{set-paper-size} ,
Drop space before comma

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacing.itely#newcode219
Documentation/notation/spacing.itely:219: which must always be placed in
a @code{\paper} block:
colon -> period

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacing.itely#newcode232
Documentation/notation/spacing.itely:232: @code{\paper} block.  See
@ref{Automatic scaling to paper size}.
I don't think we need all these @emphs; too many and
it looks like we're shouting at the reader :)

http://codereview.appspot.com/5783075/



reply via email to

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