[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fixes position of mensural c clef (issue 6503091)
From: |
PhilEHolmes |
Subject: |
Re: Fixes position of mensural c clef (issue 6503091) |
Date: |
Tue, 25 Sep 2012 15:18:27 +0000 |
On 2012/09/25 14:04:03, lemzwerg wrote:
LGTM, except one small issue.
http://codereview.appspot.com/6503091/diff/20001/mf/parmesan-clefs.mf
File mf/parmesan-clefs.mf (right):
http://codereview.appspot.com/6503091/diff/20001/mf/parmesan-clefs.mf#newcode816
mf/parmesan-clefs.mf:816: 2.2 reduced_il#);
The bbox is still too small.
Agreed. Will fix this in the next patch. I tried a variety of ways of
displaying the bounding box, but can only seem to be able show the right
edge. Is there a way of showing the whole box?
http://codereview.appspot.com/6503091/
- Re: Fixes position of mensural c clef (issue 6503091), (continued)
- Re: Fixes position of mensural c clef (issue 6503091), PhilEHolmes, 2012/09/24
- Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/24
- Re: Fixes position of mensural c clef (issue 6503091), PhilEHolmes, 2012/09/24
- Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/24
- Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/24
- Re: Fixes position of mensural c clef (issue 6503091), PhilEHolmes, 2012/09/25
- Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/25
- Re: Fixes position of mensural c clef (issue 6503091),
PhilEHolmes <=
- Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/25