Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread pkx166h
Hope these aren't too nit-picky. http://codereview.appspot.com/3667041/diff/13001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/3667041/diff/13001/Documentation/notation/input.itely#newcode527 Documentation/notation/input.itely

Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread Carl . D . Sorensen
Mark, This is really useful stuff. Thanks! However, I agree with Trevor about the LM vs NR style of writing. The NR needs to be terse and show, rather than tell, the story. In an effort to be clear, we often try to both tell and show. I spent a lot of time when I first started writing NR sec

Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread v . villenave
Greetings Mark, I actually kinda like this writing style, it's more explanatory and detailed that other NR material but if it can help make things more clear then I think we should accept it. The downside is that it "talks through the code" quite a lot, perhaps you could remove all variable name

Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread tdanielsmusic
I've looked at the compiled version now. It's nicely written, but my concern is that this is no longer written in 'reference' style. To me, parts of it seem more suited to the LM. The idea of the NR is that people already well-familiar with LilyPond can quickly find a keyword or syntax they hav

Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread nicolas . sceaux
A short note about bookpart titles. http://codereview.appspot.com/3667041/diff/13001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/3667041/diff/13001/Documentation/notation/input.itely#newcode538 Documentation/notation/input.it

Re: Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-19 Thread tdanielsmusic
Just a few comments after eye-balling the patch. I'll need to download and compile to comment more fully. http://codereview.appspot.com/3667041/diff/13001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/3667041/diff/13001/Docume

Rewrite NR 3.2 Titles and headers. (issue3667041)

2010-12-18 Thread markpolesky
Reviewers: , Message: Hey guys. Here's a complete rewrite of NR 3.2 Titles and headers. Please comment. You can disregard the first couple of patch sets, and go straight to the "First Draft" one. Thanks. - Mark Description: Rewrite NR 3.2 Titles and headers. Please review this at http://cod