From 11a21e9285f06006e10ca84ba6d7319e72d38d85 Mon Sep 17 00:00:00 2001 From: Mike Solomon Date: Wed, 5 Sep 2012 00:52:19 +0200 Subject: [PATCH] LedgerLineSpanner included in vertical skylines of VerticalAxisGroup. An elements grob array was erroneously being created for the LedgerLinesSpanner. This has been eliminated, which now means that its stencil can be used to calculate its vertical skylines. --- input/regression/ledger-lines-dynamics.ly | 10 ++++++++++ lily/ledger-line-engraver.cc | 1 - scm/define-grobs.scm | 1 + 3 files changed, 11 insertions(+), 1 deletion(-) create mode 100644 input/regression/ledger-lines-dynamics.ly diff --git a/input/regression/ledger-lines-dynamics.ly b/input/regression/ledger-lines-dynamics.ly new file mode 100644 index 0000000000..a6e2d80c4c --- /dev/null +++ b/input/regression/ledger-lines-dynamics.ly @@ -0,0 +1,10 @@ +\version "2.17.2" + +\header { + texidoc = "Dynamics and other outside staff objects avoid ledger lines. +" +} + +\relative c' { + f16\pp[ c d e ] r2. | +} diff --git a/lily/ledger-line-engraver.cc b/lily/ledger-line-engraver.cc index 614874a8e8..2bb12c9cdc 100644 --- a/lily/ledger-line-engraver.cc +++ b/lily/ledger-line-engraver.cc @@ -95,7 +95,6 @@ Ledger_line_engraver::stop_spanner () if (span_) { span_->set_bound (RIGHT, unsmob_grob (get_property ("currentCommandColumn"))); - Pointer_group_interface::set_ordered (span_, ly_symbol2scm ("elements"), false); span_ = 0; } } diff --git a/scm/define-grobs.scm b/scm/define-grobs.scm index 5083b5053d..528fdda26b 100644 --- a/scm/define-grobs.scm +++ b/scm/define-grobs.scm @@ -1206,6 +1206,7 @@ (minimum-length-fraction . 0.25) (springs-and-rods . ,ly:ledger-line-spanner::set-spacing-rods) (stencil . ,ly:ledger-line-spanner::print) + (vertical-skylines . ,ly:grob::vertical-skylines-from-stencil) (X-extent . #f) (Y-extent . #f) (meta . ((class . Spanner) -- 2.39.5