Re: Doc: new syntax for \tweak, \override (2936) (issue 6852052)

2012-11-23 Thread dak
On 2012/11/23 23:51:13, Trevor Daniels wrote: https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-default

Re: Doc: new syntax for \tweak, \override (2936) (issue 6852052)

2012-11-23 Thread tdanielsmusic
https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-defaults.itely#newcode1918 Documentation/notation/changi

Doc: ensure all level 3 and 4 section headings have nodes (issue 6852077)

2012-11-23 Thread graham
LGTM https://codereview.appspot.com/6852077/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Doc: fix incorrect chording for fret diagram (2963) (issue 6849085)

2012-11-23 Thread graham
LGTM https://codereview.appspot.com/6849085/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Doc: add example left-aligning dynamic text (2943) (issue 6849086)

2012-11-23 Thread graham
LGTM https://codereview.appspot.com/6849086/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: new syntax for \tweak, \override (2936) (issue 6852052)

2012-11-23 Thread graham
https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): https://codereview.appspot.com/6852052/diff/14001/Documentation/notation/changing-defaults.itely#newcode1918 Documentation/notation/changi

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-23 Thread mike
On 19 nov. 2012, at 03:46, Keith OHara wrote: > On Sun, 18 Nov 2012 12:21:51 -0800, m...@mikesolomon.org > wrote: > >> if (to_boolean (me->get_property ("add-stem-support")) >> && Stem::has_interface (e)) >> skyline.set_min_height (e->extent (common_y, _Y_AXIS)[dir]); >> >> That's pseudo-co

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-23 Thread m...@mikesolomon.org
On 19 nov. 2012, at 03:46, Keith OHara wrote: > On Sun, 18 Nov 2012 12:21:51 -0800, m...@mikesolomon.org > wrote: > >> if (to_boolean (me->get_property ("add-stem-support")) >> && Stem::has_interface (e)) >> skyline.set_min_height (e->extent (common_y, _Y_AXIS)[dir]); >> >> That's pseudo-c