Doc: Learning: Use voices in the intended order.

Doc: Learning: Use voices in the intended order.
Doc: Notation: Mention how hideNotes hides beams.

[Carl, 2012-01-02 16:37:47]

LGTM

[Graham Percival, 2012-01-07 02:41:11]

this commit appears to break staging.

http://codereview.appspot.com/5507050/diff/6001/Documentation/learning/tweaks...
File Documentation/learning/tweaks.itely (right):

http://codereview.appspot.com/5507050/diff/6001/Documentation/learning/tweaks...
Documentation/learning/tweaks.itely:2899: \\
are you CERTAIN that this actually builds?  because patchy is choking on this
@lilypond.

I will remove this commit from staging.

[Graham Percival, 2012-01-07 03:20:55]

On 2012/01/07 02:41:11, Graham Percival wrote:
> this commit appears to break staging.

hmm, I'm seeing an error somewhere else even after removing this.  It's just
possible that there's some random breakage that coincidentally just happened to
occur in this location.  If that's the case, I apologize.

[Trevor Daniels, 2012-03-10 12:36:05]

I could see no reason for this patch causing problems with make or make doc -
both ran clean here.  So I split this into the two unrelated patches to Learning
and Notation, corrected the patch to Learning and pushed to staging.  I'll check
the patch to Notation next.

[Trevor Daniels, 2012-03-10 15:17:04]

The second part of this patch has now been pushed to staging, so (assuming they
make it to master) this issue can be closed.

[Keith, 2012-03-13 05:53:39]

On 2012/03/10 12:36:05, Trevor Daniels wrote:
> I could see no reason for this patch causing problems

I had mis-spelled the option to ignore-collisions, which threw a warning, but I
didn't catch it because other overrides made the ignore-collisions setting
(which you removed) redundant, so the output looked fine.

Thanks for running a make-doc and pushing.

http://codereview.appspot.com/5507050/diff/13004/Documentation/learning/funda...
File Documentation/learning/fundamental.itely (right):

http://codereview.appspot.com/5507050/diff/13004/Documentation/learning/funda...
Documentation/learning/fundamental.itely:805: \once \override NoteColumn
#'ignore-collision = ##t
The line above can be removed, because in the pushed patch it is removed in the
version in Ch 4.

http://codereview.appspot.com/5507050/diff/13004/Documentation/learning/funda...
Documentation/learning/fundamental.itely:855: \once \override NoteColumn
#'ignore-collision = ##t
Line above can be removed

[t.daniels_treda.co.uk, 2012-03-13 09:41:36]

Keith, you wrote Tuesday, March 13, 2012 5:53 AM
Subject: Re: Doc: Learning: Use voices in the intended order. (issue 
5507050)


> On 2012/03/10 12:36:05, Trevor Daniels wrote:
>> I could see no reason for this patch causing problems
>
> I had mis-spelled the option to ignore-collisions, which threw a
> warning, but I didn't catch it because other overrides made the
> ignore-collisions setting (which you removed) redundant, so the output
> looked fine.
>
> Thanks for running a make-doc and pushing.

You're welcome - as the original misleading voice ordering was mine I was 
happy to get it fixed!

Trevor