On 2011/01/25 13:57:32, Graham Percival wrote:
Pushed.
cb03a19174fd9245008176716742e1a2eb3a0b93
Thanks,
Carl
http://codereview.appspot.com/4061043/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypo
On 2011/01/25 00:20:29, Carl wrote:
On 2011/01/24 16:33:22, Graham Percival wrote:
> Almost there.
Done, and new patch set uploaded.
Compiles fine; please push.
Cheers,
- Graham
http://codereview.appspot.com/4061043/
___
lilypond-devel mailing l
On 2011/01/24 16:33:22, Graham Percival wrote:
Almost there.
Could you run makelsr and then don't forget to do
git add Documentation/snippets/*.ly
Done, and new patch set uploaded.
Thanks,
Carl
http://codereview.appspot.com/4061043/
___
li
On 1/24/11 9:33 AM, "percival.music...@gmail.com"
wrote:
> Almost there.
>
>
> http://codereview.appspot.com/4061043/diff/19001/Documentation/snippets/new/ly
> rics-old-spacing-settings.ly
> File Documentation/snippets/new/lyrics-old-spacing-settings.ly (right):
>
> http://codereview.appspo
Almost there.
http://codereview.appspot.com/4061043/diff/19001/Documentation/snippets/new/lyrics-old-spacing-settings.ly
File Documentation/snippets/new/lyrics-old-spacing-settings.ly (right):
http://codereview.appspot.com/4061043/diff/19001/Documentation/snippets/new/lyrics-old-spacing-setting
Carl, having struggled myself trying to smooth the transition to the new
spacing with a snippet(s), I appreciate the simplicity of this solution.
It looks good to me.
Having text in CHANGES seems important.
A little sugar would help the medicine go down; if spin is not your
style, one of the ot
I've responded to the comments in detail below.
Graham, relative to your comments on issues 1483 and 1486, both are
addressed in this patch.
The templates now have the lyrics attached properly to the staves, as do
the examples in the NR. This takes care of 1483.
The templates also have suffici
On 2011/01/23 09:09:09, Trevor Daniels wrote:
I think the templates would be improved with
system-system-spacing #'basic-distance = #20
added to \paper. Otherwise the bass lyrics will be too
close to the soprano lyrics on the following system.
Done. I also added top-system-spacing and last-bo
http://codereview.appspot.com/4061043/diff/1/Documentation/changes.tely
File Documentation/changes.tely (right):
http://codereview.appspot.com/4061043/diff/1/Documentation/changes.tely#newcode70
Documentation/changes.tely:70: Lyrics above a staff must have their
@code{staff-affinity} set to
I do
I think we have a convert-ly problem.
http://codereview.appspot.com/4061043/diff/1/Documentation/snippets/new/lyrics-old-spacing-settings.ly
File Documentation/snippets/new/lyrics-old-spacing-settings.ly (right):
http://codereview.appspot.com/4061043/diff/1/Documentation/snippets/new/lyrics-old
LGTM
I think the templates would be improved with
system-system-spacing #'basic-distance = #20
added to \paper. Otherwise the bass lyrics will be too
close to the soprano lyrics on the following system.
Trevor
http://codereview.appspot.com/4061043/
___
Reviewers: ,
Message:
Here is a potential patch for fixing the lyric spacing regressions.
It updates the documentation to clarify what is needed to make lyrics
behave as desired.
It also adds a snippet that demonstrates how to achieve the old spacing
behavior.
Please respond with any comments.
12 matches
Mail list logo