Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-14 Thread pkx166h
New patch http://codereview.appspot.com/6137050/diff/22001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/22001/Documentation/notation/input.itely#newcode1044 Documentation/notation/input.itely:1044: Automatically n

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-14 Thread graham
LGTM http://codereview.appspot.com/6137050/diff/22001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/22001/Documentation/notation/input.itely#newcode1044 Documentation/notation/input.itely:1044: Automatically number

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-13 Thread pkx166h
New Patch. Thanks Keith and Jean-Charles http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely#newcode1037 Documentation/notation/in

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-13 Thread lilyfan
Nitpicks, but I had to read twice and carefully the first paragraph and I'm not sure to understand. http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/14003/Documentation

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread k-ohara5a5a
http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely#newcode1075 Documentation/notation/input.itely:1075: \footnote #'(0.5 , 0.5) #'

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread k-ohara5a5a
Big change, still very clear. http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/14003/Documentation/notation/input.itely#newcode1048 Documentation/notation/input.itely:1

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread pkx166h
New patch set uploaded http://codereview.appspot.com/6137050/diff/8003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/8003/Documentation/notation/input.itely#newcode1044 Documentation/notation/input.itely:1044: Auto

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread k-ohara5a5a
On Sat, 12 May 2012 12:11:45 -0700, wrote: \breathe c2 \footnote #'(0.5 . 0.5) #'BreathingSign \markup { Breathe } Not exactly the most intuitive order I have to say. True. Good to document it, I guess. http://codereview.appspot.com/6137050/diff/8003/Documentation/not

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread dak
On 2012/05/12 19:25:34, dak wrote: On 2012/05/12 19:11:44, dak wrote: > On 2012/05/12 18:36:56, Keith wrote: > > > http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely > > File Documentation/notation/input.itely (right): > > > > > http://codereview.appspot.com/

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread dak
On 2012/05/12 19:11:44, dak wrote: On 2012/05/12 18:36:56, Keith wrote: > http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely > File Documentation/notation/input.itely (right): > > http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.i

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread dak
On 2012/05/12 18:36:56, Keith wrote: http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely#newcode1073 Documentation/notation/inp

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread k-ohara5a5a
Looks good to me. I honestly tried to use a footnote a couple months ago, but gave up trying to figure it out. Now I think the docs are pretty clear. http://codereview.appspot.com/6137050/diff/8003/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://code

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread pkx166h
See inline. One more patch will be needed. http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely#newcode1050 Documentation/notation/

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread dak
http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/11001/Documentation/notation/input.itely#newcode1050 Documentation/notation/input.itely:1050: is to be attached but in t

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-12 Thread pkx166h
New patch set uploaded for the <> change. http://codereview.appspot.com/6137050/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-04 Thread Carl . D . Sorensen
On 2012/05/04 20:21:14, t.daniels_treda.co.uk wrote: wrote Friday, May 04, 2012 8:43 PM > > I think that makes more sense to explain. Definitely! I totally agree. Only allow articulation footnotes. Much clearer. Make a convert-ly rule that gives a warning if you ha

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-04 Thread Trevor Daniels
wrote Friday, May 04, 2012 8:43 PM Ok, everybody will love me for my late thinking again. But here we go: :) we have two cases of footnotes: one that will attach itself to whatever happens at a given point of time, being an independent event. I propose we _always_ let them try to attach

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-04 Thread dak
Ok, everybody will love me for my late thinking again. But here we go: we have two cases of footnotes: one that will attach itself to whatever happens at a given point of time, being an independent event. Those footnotes, having a natural duration of 0, come (unless they are in simultaneous musi

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-04 Thread pkx166h
http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely#newcode1050 Documentation/notation/input.itely:1050: footnote is being attached but can

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-04 Thread tdanielsmusic
Hi James Alternative text suggested. Trevor http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely#newcode1050 Documentation/notation/input.

Re: Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-05-01 Thread pkx166h
Reviewers: Graham Percival, http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely#newcode1050 Documentation/notation/input.itely:1050: footn

Doc: NR clarified \footnote command as a TextScript (issue 6137050)

2012-04-30 Thread graham
http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): http://codereview.appspot.com/6137050/diff/1/Documentation/notation/input.itely#newcode1050 Documentation/notation/input.itely:1050: footnote is being attached but can