[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Time signature before repeat bars at beginning of line; issue 828 (i
From: |
k-ohara5a5a |
Subject: |
Re: Time signature before repeat bars at beginning of line; issue 828 (issue 8508043) |
Date: |
Mon, 08 Apr 2013 17:13:57 +0000 |
Reviewers: lemzwerg,
Message:
On 2013/04/08 06:35:48, lemzwerg wrote:
LGTM. Do we have a regtest for this?
Yes, because several tests already have repeat bars at the start of
lines. They look noticeably odd before this patch, so I think they can
serve to alert us if this patch ever fails in the future.
Description:
Time signature before repeat bars at beginning of line; issue 828
Please review this at https://codereview.appspot.com/8508043/
Affected files:
M scm/define-grobs.scm
Index: scm/define-grobs.scm
diff --git a/scm/define-grobs.scm b/scm/define-grobs.scm
index
e825cb1ef0a8c4f3192dea85c08d79234c3ee0e3..0880124ecb1a3d41538c07ba956dae2103660c12
100644
--- a/scm/define-grobs.scm
+++ b/scm/define-grobs.scm
@@ -481,8 +481,8 @@
clef
key-cancellation
key-signature
- staff-bar
time-signature
+ staff-bar
cue-clef
custos)))
(non-musical . #t)
@@ -2454,7 +2454,7 @@
(cue-clef . (extra-space . 1.5))
(first-note . (fixed-space . 2.0))
(right-edge . (extra-space . 0.5))
- (staff-bar . (minimum-space . 2.0))))
+ (staff-bar . (extra-space . 1.0))))
(stencil . ,ly:time-signature::print)
(Y-extent . ,grob::always-Y-extent-from-stencil)
(style . C)