From: David Kastrup <dak@gnu.org>
Date: Thu, 26 Dec 2013 15:07:50 +0000 (+0100)
Subject: Issue 621: Dynamics should avoid cross-staff BarLines
X-Git-Tag: release/2.19.0-1~18
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=9edf3715c2b2cca09785dcd015724ad3c29cba8a;p=lilypond.git

Issue 621: Dynamics should avoid cross-staff BarLines
(e.g. GrandStaff, PianoStaff etc)

Dynamics usually have extra-spacing-width set to an empty interval so
that their placement does not cause other elements to shift.

With span bars, however, the resulting overlap is a worse cure than
the problem.  So this switches off the width-hiding setting of
extra-spacing-width inside of staff groups using span bars by default.
No extra space is allocated, so dynamics will clear the span bars only
narrowly, a reasonable compromise.
---

diff --git a/ly/engraver-init.ly b/ly/engraver-init.ly
index dfaa47e12f..16ad8c84ef 100644
--- a/ly/engraver-init.ly
+++ b/ly/engraver-init.ly
@@ -313,6 +313,9 @@ contained staves are connected vertically."
 
   \consists "Instrument_name_engraver"
   \consists "Span_bar_engraver"
+% The default for DynamicText.extra-spacing-width causes dynamics to
+% be placed across span bars, so switch it off:
+  \override DynamicText.extra-spacing-width = ##f
   \consists "Span_bar_stub_engraver"
   \consists "Span_arpeggio_engraver"
   \consists "System_start_delimiter_engraver"
@@ -359,6 +362,9 @@ together, never separately."
 
   \consists "Instrument_name_engraver"
   \consists "Span_bar_engraver"
+% The default for DynamicText.extra-spacing-width causes dynamics to
+% be placed across span bars, so switch it off:
+  \override DynamicText.extra-spacing-width = ##f
   \consists "Span_bar_stub_engraver"
   \consists "Span_arpeggio_engraver"
   \consists "Output_property_engraver"