[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: |
Mon, 24 Sep 2012 17:34:36 +0000 |
The glyph shape is OK, thanks. Have you tested the appearance on paper
of all available sizes, using a 300 or 600dpi printer?
http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf
File mf/parmesan-clefs.mf (left):
http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#oldcode902
mf/parmesan-clefs.mf:902:
The dimension of the bounding box is not correct. It's far too large,
as can be seen in the DVI proof.
http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf
File mf/parmesan-clefs.mf (right):
http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#newcode791
mf/parmesan-clefs.mf:791: ..z2+(0, blot_rad)
This must be `.. {up}z2 + (0, blot_rad)', otherwise the curve doesn't
start with the right direction. This wouldn't be that bad, but it were
different to all other glyph shapes.
Ditto for all other transitions from `--' to `..' (this is, using the
proper `{up}', `{right}', etc.).
http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#newcode793
mf/parmesan-clefs.mf:793: --quartercircle scaled blot_diameter rotated
180 shifted (z3r + (blot_rad, blot_rad))
Using `quartercircle' gives good results in the type1 output. I only
ask you to make the line fit into 80 columns :-)
On the other hand, why not simply using something like
startcurve_point{down} .. {right}endcurve_point
? This might be shorter, and easier to read.
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 <=
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, 2012/09/25
Re: Fixes position of mensural c clef (issue 6503091), lemzwerg, 2012/09/25