Mark, I wanted to flag a bit of cleanup, and this reasonably-related
patch still shows as 'open'.


http://codereview.appspot.com/2758042/diff/1/Documentation/notation/spacing.itely
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/2758042/diff/1/Documentation/notation/spacing.itely#newcode1050
Documentation/notation/spacing.itely:1050: \override VerticalAxisGroup
#'minimum-Y-extent = #'(-6 . 6)
The minimum-Y-extent of a VerticalAxisGroup no longer has effect, by my
test, which was prompted by Mark's catch of the convert-ly bug in re
issue 1299. (Other objects still use minimum-Y-extent.)  Suggest simply
cutting this one \context block, because your other doc patch (line
1684) contains an example showing the new spacing method in better
context, and re-enforces the use of \layout.

http://codereview.appspot.com/2758042/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to