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/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to