Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(117)

Issue 4748044: font: change breve vertical lines

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 9 months ago by Janek Warchol
Modified:
12 years, 9 months ago
Reviewers:
lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

font: change breve vertical lines Put breve vertical lines farther apart and make them longer to increase readability.

Patch Set 1 #

Total comments: 2

Patch Set 2 : gap smaller, height bigger #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -16 lines) Patch
M mf/feta-noteheads.mf View 1 5 chunks +33 lines, -16 lines 0 comments Download

Messages

Total messages: 5
Janek Warchol
Hi, breve glyphs in Feta font need modification. What is bad now: breves lying between ...
12 years, 9 months ago (2011-07-17 10:22:13 UTC) #1
lemzwerg
http://codereview.appspot.com/4748044/diff/1/mf/feta-noteheads.mf File mf/feta-noteheads.mf (right): http://codereview.appspot.com/4748044/diff/1/mf/feta-noteheads.mf#newcode181 mf/feta-noteheads.mf:181: % when there is an interval of fourth. Wouldn't ...
12 years, 9 months ago (2011-07-27 05:03:15 UTC) #2
Janek Warchol
A pdf proof file is attached to tracker issue http://code.google.com/p/lilypond/issues/detail?id=1767 The gap in smallest font ...
12 years, 9 months ago (2011-07-28 21:17:43 UTC) #3
lemzwerg
> Aww, i was so proud of this code... :-) > Frankly, i don't think ...
12 years, 9 months ago (2011-07-29 20:29:20 UTC) #4
Janek Warchol
12 years, 9 months ago (2011-07-30 16:36:29 UTC) #5
2011/7/29  <lemzwerg@googlemail.com>:
>> Aww, i was so proud of this code...
>
> :-)
>
>> Frankly, i don't think we will gain anything from defining
>> a global value.  The algorithm i wrote is a bit complicated,
>> but i thinks is easier to understand than what you suggest
>> (if i understood your suggestion correctly).  It keeps
>> things in one place instead of values flying all around (at
>> least that's how i see it).
>
> Just to make it clear: Let's call my hypothetical variable
> `line-height'.  Then we would have in file `feta-noteheads11.mf' the
> line
>
>  line-height := 10
>
> in file `feta-noteheads13.mf' the line
>
>  line-height := 15
>
> etc. (the values are just for demonstration purposes).  Not really
> complicated or confusing IMHO.

I understand.
However, i prefer having it in one place.  After all, it's not needed
anywhere else, nothing depends on it (unlike design_size).

> Just imagine that your algorithm yields the value 13 for size 13, but
> you want to use 15 since this looks better.  How would you deal with
> that?

On the other hand, if you wanted to make a systematic change (i.e.
increase value by 2 everywhere), using algorithm is cleaner.

> Add another exception to your algorithm, or redesign it completely?

Probably redesign.  Everything that can reasonably be described by an
algorithm should be done so (in my opinion).  If an algorithm exists,
it explains why things are the way they are; it can also be applied in
cases not originally predicted by the author.
In this case i think the algorithm approach is reasonable, but I don't
feel extremely strongly about that.

cheers,
Janek
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b