|
From: | k-ohara5a5a |
Subject: | Re: Adds padding between Hairpins and SpanBars. (issue 5438060) |
Date: | Tue, 29 Nov 2011 02:13:25 +0000 |
On 2011/11/28 09:11:47, mike_apollinemike.com wrote:
If the hairpins stop before span bars but extend all the way when span bar's don't exist (including when they are not present because of the RemoveEmptyStaffContext), then I'd much rather go with your patch, as it is much less invasive than mine.
Lemme know!
From the response on the tracker issue, it seems that people don't want the complicated behavior you just described, so we can go with the simpler <http://codereview.appspot.com/5373048/> ; it made a clean check. I linked it to issue 2060 but it fixes 2057 as well. http://codereview.appspot.com/5438060/
[Prev in Thread] | Current Thread | [Next in Thread] |