Re: Doc: NR 4.1.2: Reorganize vertical dimensions. (issue2316042)

[percival . music . ca]

LGTM, although I have no knowledge of the spacing code.


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

http://codereview.appspot.com/2316042/diff/1/Documentation/notation/spacing.itely#newcode198
Documentation/notation/spacing.itely:198: @address@hidden vertical
dimensions}}
Please use @subsubheading for this kind of stuff.

In the long term, we might even make "Vertical dimensions" into a
@subsection, then make things like "Fixed vertical dimensions",
"Flexible vertical dimensions", etc., into @unnumberedsubsubsec.  But
for this first patch, I'd rather keep on focusing on documenting the
behaviour rather than playing games with texinfo sectioning.  :)

http://codereview.appspot.com/2316042/