LGTM, with some minor issues. Thanks!
https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely File Documentation/notation/text.itely (right): https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode50 Documentation/notation/text.itely:50: @cindex Text objects Any specific reason why you use uppercase? Wouldn't @cindex text objects be sufficient? https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode51 Documentation/notation/text.itely:51: @cindex Markup objects BTW, I think you should also add @cindex objects, markup for completeness. https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode63 Documentation/notation/text.itely:63: Grammatical nitpick: If you have a list of items, either do This is as follows. * Foo bar. * Ping pong. * Frob nicate. or This is as follows: * foo bar; * ping pong; * frob nicate. https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode82 Documentation/notation/text.itely:82: in any definition inside the @code{\header} block (e.g. Mid-sentence abbreviations that end with a full dot should have a trailing '@:' to get correct whitespace. In this case, however, I would write 'e.g.,' https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode83 Documentation/notation/text.itely:83: @var{title}, @var{subtitle}, @var{composer}), or in some s/@var/@code/ Those header block elements are not variable (i.e., they cannot be changed by the user to an arbitrary string). https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode85 Documentation/notation/text.itely:85: @var{evenHeaderMarkup} for page numbers. This is explained in Ditto. https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode89 Documentation/notation/text.itely:89: @noindent I don't see a necessity for '@noindent'. You actually start a new paragraph, don't you? https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode119 Documentation/notation/text.itely:119: @noindent Ditto. https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode122 Documentation/notation/text.itely:122: @var{text} property if it has one, or its @var{stencil} property. Some of s/@var/@code/ https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode148 Documentation/notation/text.itely:148: \once \override NoteHead.text = \markup \lower #0.5 "NoteHead" Please check the PDF output; I guess that some lines in your example are too wide. https://codereview.appspot.com/357930043/diff/20001/Documentation/notation/text.itely#newcode641 Documentation/notation/text.itely:641: Extentind LilyPond: s/Extentind/Extending/ https://codereview.appspot.com/357930043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel