On 2015/11/13 21:51:19, dak wrote:
Sorry for yet finding more stuff that, after all, could likely be
improved. At
least I did so pretty quickly this time.
https://codereview.appspot.com/270640043/diff/60001/scm/define-markup-commands.scm
File scm/define-markup-commands.scm (right):
https://codereview.appspot.com/270640043/diff/60001/scm/define-markup-commands.scm#newcode623
scm/define-markup-commands.scm:623: (direction DOWN) Should this markup command be called "undertie" or should it rather be
"tie",
with "undertie" explicitly overriding `direction'?
Because the rather explicit name '\undertie' seems a bit inconsistent
with the
behavior of
{ c'1^\markup \undertie hm }
where the direction is determined by the direction specified for the
TextScript. Changed property direction to text-direction, which is in use already. Now under/overtie should be robust against direction-modifiers
https://codereview.appspot.com/270640043/diff/60001/scm/stencil.scm File scm/stencil.scm (right):
https://codereview.appspot.com/270640043/diff/60001/scm/stencil.scm#newcode86
scm/stencil.scm:86: (interval-index '(0 . 1) offset-index)) This is just (+ 0.1 (* 0.3 angularity)) I think and right-control
would then be
(- 1 left-control).
Done. https://codereview.appspot.com/270640043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel