lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Vertical spacing tutorial (Issue 2809) (issue 11966043)


From: k-ohara5a5a
Subject: Re: Vertical spacing tutorial (Issue 2809) (issue 11966043)
Date: Fri, 02 Aug 2013 07:06:13 +0000

With one change, LGTM.

I did not think it wise to even try to explain LilyPond's interface for
staff spacing in the Learning Manual, but you seem to have tried and
succeeded.


https://codereview.appspot.com/11966043/diff/1/Documentation/learning/tweaks.itely#newcode2472
> Documentation/learning/tweaks.itely:2472: property.  Spacing them
away
> from the staff which they relate to
> Sorry to be pedantic, but I'm of an age that still
> prefers to see "to which" in print.
>
> https://codereview.appspot.com/11966043/

but "up with which I will not put" is relevant?


You could argue that 'relate' is a verb used in its normal way and "to
the staves" is the implied prepositional phrase, so the "to which"
belongs together.  The French translation will probably use a single
word 'dont' for "to which".

By contrast "put up with" is a phrasal verb whose meaning does not
follow from the usual meaning of "put" "up" and "with", so we keep it
together -- and "put up with" probably gets translated into a single
word in French.

"How many children do you look after?"
"Under how many rocks did you look?"


https://codereview.appspot.com/11966043/diff/1/Documentation/learning/tweaks.itely
File Documentation/learning/tweaks.itely (right):

https://codereview.appspot.com/11966043/diff/1/Documentation/learning/tweaks.itely#newcode2431
Documentation/learning/tweaks.itely:2431: \new Score {
For some reason, people stopped using \new Score: issue 1033

https://codereview.appspot.com/11966043/diff/1/Documentation/learning/tweaks.itely#newcode2565
Documentation/learning/tweaks.itely:2565: @code{VerticalAxisGroup}, but
this time we're going to reduce both
This should be StaffGrouper again, as it was in the ChoirStaff.

Overriding the VerticalAxisGroup of each individual staff inside the
PianoStaff, as you have it now, would also close the space after the
piano staff if there were more staves below.

You might help people avoid issue 3482 if you encourage the pattern :
A) contexts that make groups of lines like GrandStaff, PianoStaff, etc.,
create StaffGroup objects.
B) contexts that make single lines like Lyrics, Staff, etc., create
VerticalAxisGroup objects.

https://codereview.appspot.com/11966043/diff/1/Documentation/learning/tweaks.itely#newcode2596
Documentation/learning/tweaks.itely:2596: \override
VerticalAxisGroup.staff-staff-spacing = #'(
\override StaffGrouper.staff-staff-spacing.basic-distance = #0

https://codereview.appspot.com/11966043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]