[Top][All Lists]
[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/