Re: Document use of alignAboveContext and alignBelowContext (issue 6353079)

2012-07-11 Thread tdanielsmusic
Patch pushed bcd9d85bfc1dd6cc2dd3e98901b4df33fd6d0989 http://codereview.appspot.com/6353079/ ___ lilypond-user mailing list lilypond-user@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-user

Re: Document use of alignAboveContext and alignBelowContext (issue 6353079)

2012-07-08 Thread graham
LGTM. Ignore my other message that talked about using git-cl to upload an edited patch; I hadn't noticed that you did that here. http://codereview.appspot.com/6353079/diff/6001/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): http://co

Re: Document use of alignAboveContext and alignBelowContext (issue 6353079)

2012-07-08 Thread tdanielsmusic
Reviewers: phileholmes_googlemail.com, Graham Percival, http://codereview.appspot.com/6353079/diff/1/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): http://codereview.appspot.com/6353079/diff/1/Documentation/notation/changing-defaults.

Re: Document use of alignAboveContext and alignBelowContext (issue 6353079)

2012-07-07 Thread graham
http://codereview.appspot.com/6353079/diff/1/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (right): http://codereview.appspot.com/6353079/diff/1/Documentation/notation/changing-defaults.itely#newcode1024 Documentation/notation/changing-default

Document use of alignAboveContext and alignBelowContext (issue 6353079)

2012-07-06 Thread PhilEHolmes
One suggestion, otherwise LGTM. Thanks. http://codereview.appspot.com/6353079/diff/1/Documentation/notation/vocal.itely File Documentation/notation/vocal.itely (right): http://codereview.appspot.com/6353079/diff/1/Documentation/notation/vocal.itely#newcode1327 Documentation/notation/vocal.itel