Menu
▾
▴
#695 slurs may take too much vertical extent
Originally created by: *anonymous
Originally created by: v.villenave
Originally owned by: mts...@gmail.com
% The text script is placed too high.
\version "2.11.62"
\relative c' {
f8^"rit"( c' f c' f) [r8] [r4]
}
% Ideally, the slur should not be treated as a box.
Originally posted by: pnorcks@gmail.com
(No comment was entered for this change.)
Owner: ---
Originally posted by: pnorcks@gmail.com
(No comment was entered for this change.)
Labels: -Engraving-nitpick
Originally posted by: ed.klari...@gmail.com
Other expressive marks have the same behavior. For example trills and dynamics.
\version "2.14.2"
\relative c''' {
% Trill
c2( c,2
g1)~\startTrillSpan
g1\stopTrillSpan
% Dynamic text
g1(\f
g,1)
}
Greetings, Ed
Originally posted by: pkx1...@gmail.com
Taken from a message on user with a workaround in this specific case that may help others who run into this (at least until this issue gets 'resolved').
--snip--
\version "2.15.21"
#(ly:set-option 'debug-skylines #t)
\relative c'' {
\override DynamicLineSpanner #'avoid-slur = #'ignore
\override DynamicTextSpanner #'avoid-slur = #'ignore
\override DynamicText #'avoid-slur = #'ignore
a\p f g e f d a g
a'(\p f g e f d a g)
}
(see attached)
Workaround suggested by Xavier S:
\once \override Slur #'Y-extent = #'(0 . 0)
is a workaround that was suggested and works quite well in this case.
http://lists.gnu.org/archive/html/bug-lilypond/2011-03/msg00501.html
--snip--
Labels: -Type-Defect -Priority-Low Type-Ugly
Originally posted by: mts...@gmail.com
Give slurs skylines in outside-staff-priority calculations.
http://codereview.appspot.com/5504055
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: mts...@gmail.com
Give slurs skylines in outside-staff-priority calculations.
http://codereview.appspot.com/5504055
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: mts...@gmail.com
Give slurs skylines in outside-staff-priority calculations.
http://codereview.appspot.com/5504055
Labels: Patch-new
Originally posted by: lilypond...@gmail.com
Patchy the autobot says: LGTM.
Labels: Patch-review
Originally posted by: ColinPKC...@gmail.com
(No comment was entered for this change.)
Labels: -Patch-review Patch-countdown
Owner: mts...@gmail.com
Originally posted by: ColinPKC...@gmail.com
Counted down to 20111224, please push
Labels: -Patch-countdown Patch-push
Originally posted by: mts...@gmail.com
Pushed to staging as [r19520fd5bbd221ca1d35011d7710e233c92a44b0]
Status: Fixed
Originally posted by: PhilEHol...@googlemail.com
(No comment was entered for this change.)
Labels: -Patch-push Fixed_2_15_24
Originally posted by: Elu...@gmail.com
all 3 examples above look good now
Status: Verified
Originally posted by: ahawry...@gmail.com
Looks great, you've made my day!
\version "2.15.25"
#(ly:set-option 'debug-skylines #t)
\relative c'' {
a\p f g e f d a g
a'(\p f g e f d a g)
}
Originally posted by: pkx1...@gmail.com
Is this worth a mention in the 'changes' tely?
Originally posted by: m...@mikesolomon.org
I'd vote for nay - I think that the only things worth mentioning in changes are things that'll potentially impact the way users use lilypond. If every spacing change were reported, changes.tely would get too long and people wouldn't pick up the most essential information.
Originally posted by: janek.li...@gmail.com
I think this will impact the way users use Lily, perhaps quite strongly (thanks, Mike!).
However i don't insist on mentioning this in changes.