[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Doc: More NR and LM additions based on user emails (issue3705042)
From: |
percival . music . ca |
Subject: |
Re: Doc: More NR and LM additions based on user emails (issue3705042) |
Date: |
Fri, 17 Dec 2010 14:05:27 +0000 |
http://codereview.appspot.com/3705042/diff/1/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):
http://codereview.appspot.com/3705042/diff/1/Documentation/learning/fundamental.itely#newcode2114
Documentation/learning/fundamental.itely:2114: objects to note heads, if
you remove the @code{Note_heads_engraver}
I'm not convinced that the comma is grammatical. How do you feel about
semicolons?
http://codereview.appspot.com/3705042/diff/1/Documentation/notation/repeats.itely
File Documentation/notation/repeats.itely (right):
http://codereview.appspot.com/3705042/diff/1/Documentation/notation/repeats.itely#newcode150
Documentation/notation/repeats.itely:150: @rprogram{An extra staff
appears}. Do not place bar checks outside
Could this be in a separate @warning? I'd rather not have two totally
distinct warnings in the same box.
I also think it's a bit wordier than it needs to be -- I really want to
keep warnings short, otherwise they lose their power. How about
"Bar checks must be placed @emph{inside} the braces of each alternate
block." ? ok, that's not quite the same thing (it says that you *must*
use bar checks)... but please try to think of a reformulation along
those lines.
http://codereview.appspot.com/3705042/
- Re: Doc: More NR and LM additions based on user emails (issue3705042),
percival . music . ca <=
- Re: Doc: More NR and LM additions based on user emails (issue3705042), Carl . D . Sorensen, 2010/12/17
- Re: Doc: More NR and LM additions based on user emails (issue3705042), pkx166h, 2010/12/17
- Re: Doc: More NR and LM additions based on user emails (issue3705042), Carl . D . Sorensen, 2010/12/17
- Re: Doc: More NR and LM additions based on user emails (issue3705042), tdanielsmusic, 2010/12/17
- Re: Doc: More NR and LM additions based on user emails (issue3705042), Carl . D . Sorensen, 2010/12/19
- Re: Doc: More NR and LM additions based on user emails (issue3705042), pkx166h, 2010/12/21
- Re: Doc: More NR and LM additions based on user emails (issue3705042), tdanielsmusic, 2010/12/21
- Re: Doc: More NR and LM additions based on user emails (issue3705042), Carl . D . Sorensen, 2010/12/21
- Re: Doc: More NR and LM additions based on user emails (issue3705042), tdanielsmusic, 2010/12/22