Re: Implement shorten-pair for Hairpin (issue 315350043 by david.nales...@gmail.com)

2017-01-14 Thread thomasmorley65
Again limited to the scm/ly-files: LGTM One nit: 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):

Re: Implement shorten-pair for Hairpin (issue 315350043 by david.nales...@gmail.com)

2017-01-14 Thread david . nalesnik
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 } {

Re: How to adjust space between ChordNames and Staff?

2017-01-14 Thread Trevor Daniels
Thomas Morley wrote Friday, January 13, 2017 9:05 PM > 2017-01-13 17:05 GMT+01:00 Trevor Daniels : >> >> Risto Vääräniemi wrote Friday, January 13, 2017 3:15 PM >> >>> On 13 January 2017 at 01:20, Trevor Daniels wrote: >>> Interesting. Does