Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-10 Thread janek . lilypond
Pushed as commit 59a842eba0f7ad78289a58a7dfa8fa786cdf11ed Author: Janek Warchoł lemniskata.bernoull...@gmail.com Date: Wed Mar 27 17:20:03 2013 +0100 Issue 2245: always align dynamics and lyrics on main notehead Until now, LyricTexts and DynamicTexts had their X-parents set to

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-05 Thread david . nalesnik
https://codereview.appspot.com/108270044/diff/40001/scm/define-grobs.scm File scm/define-grobs.scm (right): https://codereview.appspot.com/108270044/diff/40001/scm/define-grobs.scm#newcode87 scm/define-grobs.scm:87: (X-extent . ,ly:accidental-interface::width) The old code is used as an example

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-05 Thread k-ohara5a5a
https://codereview.appspot.com/108270044/diff/40001/scm/define-grobs.scm File scm/define-grobs.scm (right): https://codereview.appspot.com/108270044/diff/40001/scm/define-grobs.scm#newcode87 scm/define-grobs.scm:87: (X-extent . ,ly:accidental-interface::width) I opened a tracker item

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-04 Thread Janek Warchoł
Hi Mark, do you have any objections against putting this patch on countdown? As i wrote, this is a transitional phase: the patch improves the situation, but i intend to make more changes when it becomes clear how they should look like. best, Janek PS i've fixed whitespace/ordering issues in the

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-04 Thread markpolesky
On 2014/07/04 20:47:24, janek wrote: Hi Mark, do you have any objections against putting this patch on countdown? No objections, but I'll leave it to someone else to give the green light since I'm not qualified to evaluate the C++ code. As i wrote, this is a transitional phase: the patch

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-04 Thread Janek Warchoł
2014-07-04 23:21 GMT+02:00 markpole...@gmail.com: On 2014/07/04 20:47:24, janek wrote: Hi Mark, do you have any objections against putting this patch on countdown? No objections, but I'll leave it to someone else to give the green light since I'm not qualified to evaluate the C++ code. ok,

Re: Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-02 Thread Janek Warchoł
Hi, 2014-07-01 21:35 GMT+02:00 markpole...@gmail.com: This patch is problematic for me. Firstly, when a lyric syllable extends to the next note (as with a slur or tie), it is my understanding that the lyric text should be left-aligned with the left-most note of the chord. See Issue 3521:

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-02 Thread markpolesky
Janek Warchoł wrote: While I think that it's better to always align lyrics to the main notehead, I knew that some people would prefer to do this otherwise, so the patch allows to choose how to align LyricTexts (and DynamicTexts) Okay, this is less problematic than I thought it was, and I'm

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-02 Thread Janek Warchoł
Hi, 2014-07-02 20:58 GMT+02:00 markpole...@gmail.com: Janek Warchoł wrote: While I think that it's better to always align lyrics to the main notehead, I knew that some people would prefer to do this otherwise, so the patch allows to choose how to align LyricTexts (and DynamicTexts)

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-02 Thread Janek Warchoł
PS 2014-07-02 20:58 GMT+02:00 markpole...@gmail.com: Okay, this is less problematic than I thought it was, and I'm slowly convincing myself that for LyricText, main-notehead-alignment is better than NoteColumn-alignment. However, for DynamicText, I think NoteColumn-alignment is preferable,

Re: Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-07-01 Thread markpolesky
This patch is problematic for me. Firstly, when a lyric syllable extends to the next note (as with a slur or tie), it is my understanding that the lyric text should be left-aligned with the left-most note of the chord. See Issue 3521: http://code.google.com/p/lilypond/issues/detail?id=3521

Re: Issue 2245: always align dynamics and lyrics on main notehead (issue 108270044 by janek.lilyp...@gmail.com)

2014-06-29 Thread janek . lilypond
Hi folks, please review. @Han-Wen: in Script_engraver, you wrote (commit 286fcad779) that aligning stuff like Scripts on NoteColumns is not correct (lines 256-261). However, to me it seems that using NoteColumns as X-parents for Lyrics, Dynamics etc. is exactly the way to solve issue 2245