Doc: NR 4.4.1: Clean up property descriptions.

Doc: NR 4.4.1: Clean up property descriptions.

[Mark Polesky, 2010-11-14 05:09:09]

Just some clean-ups to NR 4.4.1 "Flexible vertical spacing
within systems".

Any objections?  Okay to push?
- Mark

[Trevor Daniels, 2010-11-14 09:13:35]

Looks a lot better.  I've indicated one detail.  Also the use of the spacing
suffix on property names looks inconsistent to me.  I've not marked these as
there may be more outside this patch, but it seems to be omitted or included
fairly randomly.  Could you check this out?

Trevor

http://codereview.appspot.com/3089042/diff/1/Documentation/notation/spacing.i...
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3089042/diff/1/Documentation/notation/spacing.i...
Documentation/notation/spacing.itely:1672: unset, for ungrouped staves and for
grouped staves that do not
Still difficult to parse.  How about two sentences: "... when it is unset. 
Applies to ungrouped staves ..."

[markpolesky_yahoo.com, 2010-11-14 09:47:40]

Trevor Daniels wrote:
> Looks a lot better.  I've indicated one detail.  Also the
> use of the spacing suffix on property names looks
> inconsistent to me.  I've not marked these as there may be
> more outside this patch, but it seems to be omitted or
> included fairly randomly.  Could you check this out?

I don't follow -- "the use of the spacing suffix on property
names..."  Could you explain what you mean by that?

Thanks.
- Mark


      

[markpolesky_yahoo.com, 2010-11-14 09:47:40]

Trevor Daniels wrote:
> Looks a lot better.  I've indicated one detail.  Also the
> use of the spacing suffix on property names looks
> inconsistent to me.  I've not marked these as there may be
> more outside this patch, but it seems to be omitted or
> included fairly randomly.  Could you check this out?

I don't follow -- "the use of the spacing suffix on property
names..."  Could you explain what you mean by that?

Thanks.
- Mark


      

[Trevor Daniels, 2010-11-14 12:50:00]

On 2010/11/14 09:47:40, markpolesky_yahoo.com wrote:

> I don't follow -- "the use of the spacing suffix on property
> names..."  Could you explain what you mean by that?

My mistake!  I think I must have misread staff-affinity.  (I should have learned
by now not to try to review things in 5 minutes before I have to rush out!)
Sorry!

Trevor

[Mark Polesky, 2010-11-14 16:32:03]

Thanks, Trevor.  Okay to push this now?

- Mark

http://codereview.appspot.com/3089042/diff/1/Documentation/notation/spacing.i...
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3089042/diff/1/Documentation/notation/spacing.i...
Documentation/notation/spacing.itely:1672: unset, for ungrouped staves and for
grouped staves that do not
On 2010/11/14 09:13:36, Trevor Daniels wrote:
> Still difficult to parse.  How about two sentences: "... when it is unset. 
> Applies to ungrouped staves ..."

Done.

[Trevor Daniels, 2010-11-14 17:03:35]

Looks OK to me.  Let's go.

Trevor

[Carl, 2010-11-14 18:15:09]

LGTM.

Carl