Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-30 Thread graham
LGTM http://codereview.appspot.com/5504100/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread Keith OHara
On Sun, 01 Jan 2012 14:33:33 -0800, wrote: I disagree. The example in the tracker is overly complicated for what (I still maintain) is easily explained. However *if* we really do need an example then the \hideNotes one is not ideal. I struggled myself with that one in the tracker, it isn't a '

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread pkx166h
See comments below, thanks. http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely#newcode1953 Documentation/notation/rhythms.itely:195

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread pkx166h
On 2012/01/01 21:31:50, Keith wrote: On 2012/01/01 15:06:05, dak wrote: > On 2012/01/01 14:56:25, Carl wrote: > > > Can we show this using a single voice with non-chorded notes and by using > > > something more 'obvious' than \hideNotes? \relative c' { \voiceTwoStyle c8 e \voiceNeutral

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread k-ohara5a5a
On 2012/01/01 15:06:05, dak wrote: On 2012/01/01 14:56:25, Carl wrote: > > Can we show this using a single voice with non-chorded notes and by using > > something more 'obvious' than \hideNotes? \relative c' { \voiceTwoStyle c8 e \voiceNeutralStyle g c \voiceTwoStyle c8 e \voiceNeutra

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread dak
On 2012/01/01 14:56:25, Carl wrote: > Can we show this using a single voice with non-chorded notes and by using > something more 'obvious' than \hideNotes? Override the beam color to red? Also \once\hideNotes might be a bit more snappy. http://codereview.appspot.com/5504100/

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread Carl . D . Sorensen
Can we show this using a single voice with non-chorded notes and by using something more 'obvious' than \hideNotes? Override the beam color to red? http://codereview.appspot.com/5504100/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2012-01-01 Thread pkx166h
Reviewers: carl.d.sorensen_gmail.com, Keith, Message: On 2012/01/01 03:08:19, Carl wrote: http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5504100/diff/1/Documentation/notation/r

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2011-12-31 Thread Carl . D . Sorensen
http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely#newcode1954 Documentation/notation/rhythms.itely:1954: The properties of a beam a

Re: Doc: NR added @knownissue for beam properties (issue 5504100)

2011-12-31 Thread k-ohara5a5a
LGTM http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5504100/diff/1/Documentation/notation/rhythms.itely#newcode1903 Documentation/notation/rhythms.itely:1903: @code{@bs{}autoBeamO