Doc: NR 4: Minor edits.

Doc: NR 4: Minor edits.

[Mark Polesky, 2010-12-02 07:21:33]

Here's a patch to clean up the NR spacing chapter a little
more.  Does everything look okay?

Thanks.
- Mark

[Trevor Daniels, 2010-12-02 08:34:52]

LGTM; just a few nitpicks

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % this \break works
indent; needs another c

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1298: c4 c  c c  |
spacing

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1316: c4 c  c c  |
do we need these two lines here?

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1556: music = { c'8 c' c' c' }
\relative

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:2227: primarily concern the amount of
vertical padding different staves
primarily affect the amount of vertical padding between the staves and sysems.

[Graham Percival (old account), 2010-12-02 09:16:21]

On 2010/12/02 08:34:52, Trevor Daniels wrote:
> LGTM; just a few nitpicks

I agree with Trevor; I noticed (almost) everything he did, and didn't notice
anything else.

[Neil Puttock, 2010-12-02 16:43:10]

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1294: { c4 c c c2          }
remove extra spaces before brace

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % this \break works
On 2010/12/02 08:34:52, Trevor Daniels wrote:
> indent; needs another c

adding another c will make the break invalid

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1312: { c4 c c c2          }
remove extra spaces before brace

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
Documentation/notation/spacing.itely:2234: specifies exact vertical positions on
the page.
can specify

(bearing in mind it's most likely to be used for alignment-distances, which
doesn't specify exact positioning)

[t.daniels_treda.co.uk, 2010-12-02 17:20:03]

Neil

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
> Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % 
> this
> \break works
> On 2010/12/02 08:34:52, Trevor Daniels wrote:
>> indent; needs another c
>
> adding another c will make the break invalid

You're right.  Clearly this example is confusing, with
at least two people from three demonstrably confused!

I think it should be split into two, one showing that
a hanging note prevents a line break, and another
showing that a tied note over a bar line doesn't.

Trevor

[t.daniels_treda.co.uk, 2010-12-02 17:20:03]

Neil

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacin...
> Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % 
> this
> \break works
> On 2010/12/02 08:34:52, Trevor Daniels wrote:
>> indent; needs another c
>
> adding another c will make the break invalid

You're right.  Clearly this example is confusing, with
at least two people from three demonstrably confused!

I think it should be split into two, one showing that
a hanging note prevents a line break, and another
showing that a tied note over a bar line doesn't.

Trevor

[Mark Polesky, 2010-12-04 04:45:07]

Hey guys, I made some requested changes.
How does this look?

- Mark

[Carl, 2010-12-04 08:29:57]

LGTM.

Carl

[Trevor Daniels, 2010-12-04 09:45:21]

LGTM, apart from the one typo.

http://codereview.appspot.com/3406041/diff/15001/Documentation/notation/spaci...
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3406041/diff/15001/Documentation/notation/spaci...
Documentation/notation/spacing.itely:1212: a measure, add an inivisible bar line
with @w{@samp{\bar ""}}:
invisible

[Graham Percival (old account), 2010-12-04 10:42:29]

LTGM

[Mark Polesky, 2010-12-04 22:15:59]

Pushed and closed.  Thanks guys.
- Mark