Re: Doc: NR 4.1: Reorganize, clarify details. (issue2758042)

[n . puttock]

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely
File Documentation/notation/input.itely (right):

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode774
Documentation/notation/input.itely:774: @code{make-footer} and
@code{make-header}, defined in
@code{make-header} and @code{make-footer}

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode776
Documentation/notation/input.itely:776: @file{ly/paper-defaults.ly} and
@file{ly/titling-init.ly}.
@file{ly/paper-defaults-init.ly}

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode782
Documentation/notation/input.itely:782: creates the actual page given
the system to put on it.
This seems to be seriously out of date; neither function exists (and
while both are present in 2.8, they're commented out).

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode793
Documentation/notation/input.itely:793: \bold \fontsize #3 \on-the-fly
#print-page-number-check-first
indent

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode794
Documentation/notation/input.itely:794: \fromproperty
#'page:page-number-string @} @}
newline for @} @}

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode796
Documentation/notation/input.itely:796: \bold \fontsize #3 \on-the-fly
#print-page-number-check-first
indent

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/input.itely#newcode797
Documentation/notation/input.itely:797: \fromproperty
#'page:page-number-string @} @}
newline for @} @}

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

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode116
Documentation/notation/spacing.itely:116: @item the
@code{set-paper-size} scheme function,
@item
the @code{set-paper-size} scheme function,

@item
...

etc.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode278
Documentation/notation/spacing.itely:278: @code{paper-height} is
@code{297 \mm} and the @code{paper-width}
297\mm

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode279
Documentation/notation/spacing.itely:279: is @code{210 \mm}.
210\mm

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode321
Documentation/notation/spacing.itely:321: If set to true, systems will
not spread vertically across the
`vertically across' sounds a bit strange to me; I think I'd prefer
something like `vertically down'.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode348
Documentation/notation/spacing.itely:348: The titles (from the
@address@hidden@}} section) are treated as a
@code{\header} block

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode438
Documentation/notation/spacing.itely:438: re-defines the variable:
redefines

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode555
Documentation/notation/spacing.itely:555: @item paper-width
@item
paper-width

etc.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode646
Documentation/notation/spacing.itely:646: @item two-sided
@item
two-sided

etc.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode748
Documentation/notation/spacing.itely:748: @item max-systems-per-page
@item
max-systems-per-page

etc.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode901
Documentation/notation/spacing.itely:901: in the output.  Normally only
the piece and opus header variables
@code{piece} and @code{opus}

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode972
Documentation/notation/spacing.itely:972: @item the
@code{layout-set-staff-size} scheme function,
@item
the @code{layout-set-staff-size} scheme function,

etc.

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode985
Documentation/notation/spacing.itely:985: @item @code{ragged-right}
@item
@code{ragged-right}

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode997
Documentation/notation/spacing.itely:997: \context @{ \Staff
\context @{
  \Staff

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode998
Documentation/notation/spacing.itely:998: \override VerticalAxisGroup
#'minimum-Y-extent = #'(-6 . 6)
replace

http://codereview.appspot.com/2758042/diff/35001/Documentation/notation/spacing.itely#newcode1000
Documentation/notation/spacing.itely:1000: \context @{ \Voice
\context @{
  \Voice

http://codereview.appspot.com/2758042/diff/35001/ly/paper-defaults-init.ly
File ly/paper-defaults-init.ly (right):

http://codereview.appspot.com/2758042/diff/35001/ly/paper-defaults-init.ly#newcode54
ly/paper-defaults-init.ly:54: top-margin-default = 5 \mm     % scaled to
paper-size
5\mm

etc.

http://codereview.appspot.com/2758042/diff/35001/ly/paper-defaults-init.ly#newcode59
ly/paper-defaults-init.ly:59: % TODO: Can these two be removed now? -mp
yes

http://codereview.appspot.com/2758042/diff/35001/ly/paper-defaults-init.ly#newcode141
ly/paper-defaults-init.ly:141: '((font-family . feta) (font-encoding .
fetaMusic)))
indent

http://codereview.appspot.com/2758042/