Re: Allow a markup to replace the default LyricHyphen (issue 325470043 by knup...@gmail.com)

2017-09-17 Thread dak
https://codereview.appspot.com/325470043/diff/20001/lily/lyric-hyphen.cc File lily/lyric-hyphen.cc (right): https://codereview.appspot.com/325470043/diff/20001/lily/lyric-hyphen.cc#newcode126 lily/lyric-hyphen.cc:126: Stencil dash_mol (Lookup::round_filled_box (b, 0.8 * lt)); On 2017/09/17 10:16

Re: Allow a markup to replace the default LyricHyphen (issue 325470043 by knup...@gmail.com)

2017-09-17 Thread dak
On 2017/09/17 10:02:41, knupero wrote: On 2017/09/16 19:53:57, dak wrote: > Indepent of those: I don't actually see this addressing issue 1255, so maybe > create a new issue in the Sourceforge issue tracker for it and attach this > Rietveld issue? It seems at best loosely related to issue

Re: Allow a markup to replace the default LyricHyphen (issue 325470043 by knup...@gmail.com)

2017-09-17 Thread knupero
https://codereview.appspot.com/325470043/diff/20001/lily/lyric-hyphen.cc File lily/lyric-hyphen.cc (right): https://codereview.appspot.com/325470043/diff/20001/lily/lyric-hyphen.cc#newcode125 lily/lyric-hyphen.cc:125: Box b (Interval (0, dash_length), Interval (h, h + th)); On 2017/09/16 19:53:5

Re: Allow a markup to replace the default LyricHyphen (issue 325470043 by knup...@gmail.com)

2017-09-17 Thread knupero
On 2017/09/16 19:53:57, dak wrote: Indepent of those: I don't actually see this addressing issue 1255, so maybe create a new issue in the Sourceforge issue tracker for it and attach this Rietveld issue? It seems at best loosely related to issue #1255 but seems to be desirable nevertheless