Re: adding \lyricmode to \lyricsto triggers an error

[Janek Warchoł]

2014-02-16 23:58 GMT+01:00 Eluze <address@hidden>:
> \version "2.18.0"
> % this doesn't work since 2.19.2
> <<
>   \new Voice=A \relative c'{ c d e f g a }
>   \new Lyrics \lyricsto A \lyricmode { ahh b c d e f }
>>>
> the doc says: *The lyrics themselves follow the \lyricsto command. The
> \lyricsto command invokes lyric mode automatically, so the \lyricmode
> keyword may be omitted."
>
> and indeed, omitting \lyricmode works and also putting braces after
> /\lyricsto A/ works - but I don't think this is an improvement in
> user-friendliness...

AFAIK this was changed by David K recently to make some internal stuff
more consistent. (i think it's commit
940767af99f39411e3e519a75968be11d47e3083)

I don't have an opinion about the change, but i have one suggestion:
it would be much better if objections like this were raised during
patch review.  So, please look at the review issue of this commit and
tell us what would make it easier for you to comment on it during
review instead of now.  I believe that we should adjust our policies
to encourage people like you (i.e. people who aren't core developers)
to review patches (for example from the User Interface point of view).

best,
Janek