lilypond-devel
[Top][All Lists]
Advanced

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

Re: Changes.tely updated - 2.19.x up to June 2014 (issue 108130043 by ad


From: James
Subject: Re: Changes.tely updated - 2.19.x up to June 2014 (issue 108130043 by address@hidden)
Date: Sat, 05 Jul 2014 07:08:40 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0

On 04/07/14 08:17, address@hidden wrote:
>
>
> https://codereview.appspot.com/108130043/diff/90001/Documentation/changes.tely#newcode151
>
> Documentation/changes.tely:151: \fill-line {Ooooo oOooo ooOoo oooOo
> ooooO}
> I prefer this:
> \fill-line \underline {"↓    " " ↓   " "  ↓  " "   ↓ " "    ↓"}
>
> https://codereview.appspot.com/108130043/diff/90001/Documentation/changes.tely#newcode156
>
> Documentation/changes.tely:156: \justify-line {Ooooo Ooooo Ooooo Ooooo
> Ooooo}
> I prefer this:
> \justify-line \underline {"↓    " "↓    " "↓    " "↓    " "↓    "}
>
> https://codereview.appspot.com/108130043/

Hmm...

I see what you are trying to do, but it doesn't really look that great
on the output I think. I'm not convinced.

See attached.

The arrows don't seem to show up in the PDF output (that may be my
viewer I haven't checked elsewhere) however if we did do this we would
have remove the @lilypond verbatim because you cannot really see the
'words' clearly in the construction, it's just a 'bunch' of double quotes.

I'll defer to the consensus of the dev group though.

James

Attachment: Screenshot.png
Description: PNG image


reply via email to

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