[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Doc: NR accidental styles (issue4149045)
From: |
percival . music . ca |
Subject: |
Re: Doc: NR accidental styles (issue4149045) |
Date: |
Thu, 24 Feb 2011 06:32:25 +0000 |
It's still not showy enough for me.
http://codereview.appspot.com/4149045/diff/9001/Documentation/notation/pitches.itely
File Documentation/notation/pitches.itely (right):
http://codereview.appspot.com/4149045/diff/9001/Documentation/notation/pitches.itely#newcode2527
Documentation/notation/pitches.itely:2527: Several accidental styles,
@code{modern} through @code{teaching}
Why are you telling me this?
If you want me to know that some style suppresses printing an extra
natural, then show me in the output of the @lilypond.
http://codereview.appspot.com/4149045/diff/9001/Documentation/notation/pitches.itely#newcode2530
Documentation/notation/pitches.itely:2530: For different behavior,
@code{extraNatural} may be set as desired,
Never ever say "@code{blah} may be set".
Let's get the patch pushed with showing the accidentals without extra
naturals and without any discussion of extraNatural. Then, in a second
patch, let's have a snippet that shows how to force extra natural signs
to appear. You could either do this in LSR itself, or in
Documentation/snippets/new/.
http://codereview.appspot.com/4149045/