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/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel