Re: Implement shorten-pair for Hairpin (issue 315350043 by address@hidden)

[thomasmorley65]

On 2017/01/15 17:07:57, david.nalesnik wrote:

https://codereview.appspot.com/315350043/diff/20001/Documentation/snippets/new/moving-the-ends-of-hairpins.ly
> File Documentation/snippets/new/moving-the-ends-of-hairpins.ly
(right):


https://codereview.appspot.com/315350043/diff/20001/Documentation/snippets/new/moving-the-ends-of-hairpins.ly#newcode18
> Documentation/snippets/new/moving-the-ends-of-hairpins.ly:18: c'1~\<
> On 2017/01/14 21:53:25, thomasmorley651 wrote:
> > I'd suggest to make it more clear what happens
> > if the tweaked hairpin spans between DynamicText like
> >   c'1~\p\< c'2~ c'\ffff\!
> > Maybe extend the 'hairpin'-example a bit or drop an additional line
in the
> > texidoc-header about affecting the _visible_ length of the Hairpin,
opposed to
> > the behaviour of 'minimum-length

> Done.

Thanks.

LGTM

https://codereview.appspot.com/315350043/