Re: Doc: NR rewrite of 3.2 Titles and Headers (issue4124056)

[percival . music . ca]

comments about the first section.


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

http://codereview.appspot.com/4124056/diff/32001/Documentation/notation/input.itely#newcode674
Documentation/notation/input.itely:674: @example
We just spent a month working on a8landscape, precisely to avoid having
an @example like this.  Change to @lilypond.

http://codereview.appspot.com/4124056/diff/32001/Documentation/notation/input.itely#newcode719
Documentation/notation/input.itely:719: clickable link
(@url{http://www.lilypond.org}).
Don't talk about it.  Having the example already shows you the default
tagline.

http://codereview.appspot.com/4124056/diff/32001/Documentation/notation/input.itely#newcode730
Documentation/notation/input.itely:730: c4 d e f
IIRC this produces two lines (I haven't compiled it).  If so, do we need
two lines?  Why not just have one bar?

Also, do we need a title?  This is an example for a tagline, right?

http://codereview.appspot.com/4124056/diff/32001/Documentation/notation/input.itely#newcode748
Documentation/notation/input.itely:748: c4 d e f
This is good, but previous comments apply as well.  Lose the title and
one bar.

http://codereview.appspot.com/4124056/diff/32001/Documentation/notation/input.itely#newcode755
Documentation/notation/input.itely:755: To remove the @code{tagline} set
the value to to @code{##f}.
this is fine; no need to another @lilypond here.

http://codereview.appspot.com/4124056/