Menu
▾
▴
1 Attachments
#3501 spanners fail to avoid cross-staff beams
Originally created by: *anonymous
Originally created by: k-ohara5...@oco.net
Originally owned by: k-ohara5...@oco.net
With the patch for issue 2527, some spanners began to be marked as 'cross-staff', like the Pedal markings below.
They do not actually participate in placement of other cross-staff items, but suffer the usual collisions of cross-staff objects.
<< \new Staff="up" s1
\new Staff="down"
<<{ \stemDown b8 \change Staff = "up" b b \change Staff="down" b g'2}
{ %%%\override Staff.SustainPedalLineSpanner #'cross-staff = ##f
s4\f\> s\sustainOn s\pp s\sustainOff } >>
\new Staff = "pedal" {[r4] c'''r r }>
Setting cross-staff=##f as in the %%comment above improves the output to the state we had in version 2.16.
Originally posted by: mts...@gmail.com
If you un-cross-staffify all spanners of this ilk (horizontal brackets, pedals, ottava, etc.) in your patch to 3633 & co., what does the output look like?
Originally posted by: k-ohara5...@oco.net
Spanners that belong to a Staff will not be marked cross-staff, the pedal will avoid the dynamics, and the next staff will move to clear its note-head from the pedal spanner --- same output as in 2.16 or with the workaround.
Originally posted by: k-ohara5...@oco.net
http://codereview.appspot.com/13082043
Labels: Patch-new
Originally posted by: pkx1...@gmail.com
Patchy the autobot says: passes tests.
Labels: -Patch-new Patch-review
Originally posted by: pkx1...@gmail.com
Patch on Countdown for August 24th - 06:00 GMT
Labels: -Patch-review Patch-countdown
Owner: k-ohara5...@oco.net
Originally posted by: pkx1...@gmail.com
Patch counted down - please push
Labels: -Patch-countdown Patch-push
Originally posted by: k-ohara5...@oco.net
Best to push after the fixes for 3385 and 3503, because without those fixes, this change can increase the number of "programming-error" messages for some input.
Status: Started
Originally posted by: k-ohara5...@oco.net
commit [r32b31ea446f6724b42e360f9b53a5641399b2ea6]
Labels: -Patch-push fixed_2_17_26
Status: Fixed
Originally posted by: k-ohara5...@oco.net
This might be awkward to verify.
The input at the top of the issue should now give the output we had in version 2.16:
the beam still overlaps the dynamics and the pedal * is badly aligned (issue 2261)
but at least the pedal, dynamics, and note-heads stay clear of each other.
Originally posted by: Elu...@gmail.com
(No comment was entered for this change.)
Status: Verified