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

Reply via email to