[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: |
lemzwerg |
Subject: |
Re: Fixes position of mensural c clef (issue 6503091) |
Date: |
Tue, 25 Sep 2012 14:04:03 +0000 |
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. I suggest that you define z2 like this:
x2l = 0;
top y2 = 2.2 reduced_il;
This makes the left vertical element a bit shorter which shouldn't be
noticeable, however.
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 <=
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