[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)
From: |
tdanielsmusic |
Subject: |
Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056) |
Date: |
Sun, 29 Apr 2012 09:18:28 +0000 |
LGTM, apart from the deleted index entries
http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.itely
File Documentation/notation/rhythms.itely (right):
http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.itely#newcode1415
Documentation/notation/rhythms.itely:1415: @cindex beams, unmetered
music
The change you've made was not the one I wanted. There are already
several adjacent entries in the index from 'cadenza' and 'unmetered',
but now there are none from 'bar lines' and 'bar numbers'. There should
be. If you don't like 'turning off', how about simply
@cindex bar lines, cadenzas
@cindex bar numbers, cadenzas
ditto for unmetered music
to go with the ones for beams and accidentals?
http://codereview.appspot.com/5976056/