[Top][All Lists]

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

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

From: david . nalesnik
Subject: Re: Implement shorten-pair for Hairpin (issue 315350043 by address@hidden)
Date: Sat, 14 Jan 2017 13:03:43 -0800

On 2017/01/08 11:37:18, thomasmorley651 wrote:
On 2017/01/07 17:12:57, david.nalesnik wrote:
> Please review. Thanks!

Hi David,

very nice.

I can't review C++-code, but applied the patch and tested with:

\layout { \override Hairpin.layer = 200 \override Hairpin.color = #red

   \override Hairpin.minimum-length = 40
   \override Hairpin.shorten-pair = #'(-4 . -4)
   c'1\pppp~\< c'1~ c'2~ c'2\ffff\!
   c'1~\< c'1~ c'2~ c'2\!


Obviously the actual printed extents of the Hairpin are affected.
It's now possible that the Hairpin collides with DynamicText for bad
settings of
DynamicText is not pushed out of the way. So shorten-pair is _not_ the
tool to set the overall length of a Hairpin, but for fine tunings. For
purpose minimum-length is still the way to go.

This should be documented.

Worth a regtest and an entry in changes.

Done.  Thanks again!

reply via email to

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