From 08a099c3829fea6b8192a1073347aee4ef07519f Mon Sep 17 00:00:00 2001 From: Graham Percival Date: Thu, 10 Jun 2010 22:07:24 +0100 Subject: [PATCH] Doc: revert bad fix for 617. Neil said "This is not recommended at all. The default callback is a closure, combining the result of ly:self-alignment-interface:x-aligned-on-self *and* ly:break-alignable-interface::self-align-callback. Thus overriding X-offset also causes 'break-align-symbols to be ignored, messing up the positioning completely." I don't understand it, but I understand that he knows more than me. :) I've left the little clean-up that James did. --- Documentation/notation/changing-defaults.itely | 7 ------- 1 file changed, 7 deletions(-) diff --git a/Documentation/notation/changing-defaults.itely b/Documentation/notation/changing-defaults.itely index 963b58f71b..95b394b93d 100644 --- a/Documentation/notation/changing-defaults.itely +++ b/Documentation/notation/changing-defaults.itely @@ -3421,13 +3421,6 @@ e1 e1 @end lilypond -@knownissues -Overriding the @code{#'X-offset} property to a fixed value causes -the @code{#'self-alignment-X} property to be disregarded. Therefore, -it is recommended to override it with a callback to -@code{ly:self-alignment-interface::x-aligned-on-self}, similarly to -its default definition. This method is described in -@rextend{Callback functions} @node Vertical grouping of grobs @subsection Vertical grouping of grobs -- 2.39.5