Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

[k-ohara5a5a]

Mark,
Too long, but hard to say what will be useful without getting away for a
while.  Looks good to me, whether you either take or leave my
suggestions.


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

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode233
Documentation/notation/spacing.itely:233: Inter-system spacing is
controlled by grob properties, with
Should be "Within-system"  (Inter-system would be system-system-spacing)

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode1502
Documentation/notation/spacing.itely:1502: non-staff line is its highest
point.
Suggest you stop after "its middle line."  Ref pt for Lyrics seems to be
lowest point, not sure for chords, and users will experiment anyway.

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode1528
Documentation/notation/spacing.itely:1528: To change any spacing
settings globally, put them in the
"throughout a score" instead of "globally"

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode1540
Documentation/notation/spacing.itely:1540: The global defaults for the
following grob properties are defined
if you say "defaults for StaffGrouper and VerticalAxisGroup are" then
you can skip the list.

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode1572
Documentation/notation/spacing.itely:1572: staff-group, it is usually
best to leave this property unset,
Let's be honest here " ... it is usually best to leave this property at
its default, which is a function that chooses the appropriate spacing
from the StaffGrouper object."

http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode1584
Documentation/notation/spacing.itely:1584: staves.  Does not apply to
the bottom staff of a system.
Suggest : "The value used for next-staff-spacing, if you did not
override next-staff-spacing explicitly, for a staff that is not in a
group (or for which StaffGrouper does not define a spacing)."

http://codereview.appspot.com/2642043/