[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue462907
From: |
pkx166h |
Subject: |
Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073) |
Date: |
Sun, 26 Jun 2011 12:18:25 +0000 |
Second Draft - Thanks for the comments Trevor
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely
File Documentation/notation/simultaneous.itely (right):
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode775
Documentation/notation/simultaneous.itely:775: @code{Voice} is printed.
Stem directions set up & down accordingly while
On 2011/06/26 08:58:43, Trevor Daniels wrote:
Stem directions are set automatically while Solo and @notation{a due}
parts are
identified and marked accordingly.
Done. Although changed one of the 'accordinly's to 'appropriately' just
to make it read better.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode814
Documentation/notation/simultaneous.itely:814: Even though both parts
have identical notes in the third measure, only
On 2011/06/26 08:58:43, Trevor Daniels wrote:
Both parts have identical notes in the third measure, so only one ...
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode816
Documentation/notation/simultaneous.itely:816: are set automatically,
depending whether there is a solo or unison.
On 2011/06/26 08:58:43, Trevor Daniels wrote:
depending on whether the parts are playing solo or in unison.
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode824
Documentation/notation/simultaneous.itely:824: @code{Voice} contexts, so
if using @code{\relative} octaves, then
On 2011/06/26 08:58:43, Trevor Daniels wrote:
so if the music is being specified in relative mode ...
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode834
Documentation/notation/simultaneous.itely:834: A @code{\relative}
section that is placed outside of @code{\partcombine}
On 2011/06/26 08:58:43, Trevor Daniels wrote:
section that encloses @code{\partcombine} has no ...
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode854
Documentation/notation/simultaneous.itely:854: into a chord, or showing
one voice as solo is, therefore, not ideal as
On 2011/06/26 08:58:43, Trevor Daniels wrote:
space
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode864
Documentation/notation/simultaneous.itely:864: @code{\partcombineApart}
or @code{\partcombineApartOnce}: This keeps the
On 2011/06/26 08:58:43, Trevor Daniels wrote:
I think the items in this section read better if "and" is used
together with
plural verbs, like this:
@code{\partcombineApart} and @code{\partcombineApartOnce} keep the
notes ...
Similarly for the other items (not marked).
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode956
Documentation/notation/simultaneous.itely:956: started when combining
notes that have just started in the other
On 2011/06/26 08:58:43, Trevor Daniels wrote:
space
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode957
Documentation/notation/simultaneous.itely:957: @code{Voice}. This can
lead to a number of unexpected issues including
On 2011/06/26 08:58:43, Trevor Daniels wrote:
space
Done.
http://codereview.appspot.com/4629073/diff/1/Documentation/notation/simultaneous.itely#newcode966
Documentation/notation/simultaneous.itely:966: give the voices,
internally, its own custom names; @code{one} and
On 2011/06/26 08:58:43, Trevor Daniels wrote:
colon
Done.
http://codereview.appspot.com/4629073/
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), tdanielsmusic, 2011/06/26
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073),
pkx166h <=
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), percival . music . ca, 2011/06/26
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), tdanielsmusic, 2011/06/26
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), pkx166h, 2011/06/27
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), percival . music . ca, 2011/06/27
- Re: [PATCH]: Doc: NR @knownissue for partCombine + spanners (issue4629073), pkx166h, 2011/06/28