Re: Changes to Default spacing settings (issue3061041)

2010-11-13 Thread Carl . D . Sorensen
I've posted a patch with the new spacing issues to reitveld. http://codereview.appspot.com/3099041 Keith, could you please check it to make sure it's right? Thanks, Carl http://codereview.appspot.com/3061041/ ___ lilypond-devel mailing list

Re: Changes to Default spacing settings (issue3061041)

2010-11-12 Thread percival . music . ca
oops, we were missing the CC to lilypond-devel. The patch looks generally ok to me, but I'm not up to speed on spacing stuff. Keith: why are you removing the #'stretchability = 5 ? What's the default value if you remove it? James: I suggest that the next version of the patch uses the same

Re: Changes to Default spacing settings (issue3061041)

2010-11-12 Thread k-ohara5a5a
http://codereview.appspot.com/3061041/diff/1/ly/engraver-init.ly File ly/engraver-init.ly (left): http://codereview.appspot.com/3061041/diff/1/ly/engraver-init.ly#oldcode322 ly/engraver-init.ly:322: \override StaffGrouper #'between-staff-spacing #'stretchability = #5 On 2010/11/12 14:55:28,