Doc: NR add predefined paper sizes to appendix

Doc: NR add predefined paper sizes to appendix

Issue 2391

Copied predefined paper size entries from scm/paper.scm as a seperate
appendix entry in order to better document predefined paper sizes

[dak, 2012-03-11 20:29:02]

http://codereview.appspot.com/5783075/diff/1/Documentation/notation/notation-...
File Documentation/notation/notation-appendices.itely (right):

http://codereview.appspot.com/5783075/diff/1/Documentation/notation/notation-...
Documentation/notation/notation-appendices.itely:450: @verbatim
@table @code
@item "a10"
26 x 37 mm
@item "a9"
...
@end table

You can make it prettier with a @multitable, but it will be more typing.

[pkx166h, 2012-03-17 22:49:46]

My new ammended comment in the git format patch seems to not have been included
with git-cl

--snip--
Issue 2391

Copied predefined paper size entries from scm/paper.scm as a seperate
appendix entry in order to better document predefined paper sizes.

Also revamped section on Paper Sizes including better @cindex entries,
reworded explanation and examples including how to create a custom
paper size without the need to edit scm/paper.scm

Added note in scm/paper.scm to say that the appendix entry was not
autogenerated so that any new entries will not be forgotten to be added
in the appendix - until someone 'scripts' it.

--snip--

If this is too large a patch now I can split it up.

It passes make and a full make doc.

[Trevor Daniels, 2012-03-18 17:54:02]

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/spacin...
File Documentation/notation/spacing.itely (right):

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

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacin...
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/spacin...
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 :)

[pkx166h, 2012-03-18 20:05:55]

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

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:211: @code{set-paper-size} ,
On 2012/03/18 17:54:02, Trevor Daniels wrote:
> Drop space before comma

Done.

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:219: which must always be placed in a
@code{\paper} block:
On 2012/03/18 17:54:02, Trevor Daniels wrote:
> colon -> period

Done.

http://codereview.appspot.com/5783075/diff/3001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:232: @code{\paper} block.  See
@ref{Automatic scaling to paper size}.
On 2012/03/18 17:54:02, Trevor Daniels wrote:
> I don't think we need all these @emphs; too many and
> it looks like we're shouting at the reader :)

Done.