EB-Garamond icon indicating copy to clipboard operation
EB-Garamond copied to clipboard

[Initials] Add M

Open astoeriko opened this issue 4 years ago • 5 comments

This pull request addresses part of https://github.com/georgd/EB-Garamond/issues/115 by adding the svg files for the M initial that I created from this original. I was able to build the otf files but not the rest of the font files. I think the problem is that I do not have all the required software installed to run all of the build script successfully. (I had a hard time finding out what is needed, maybe some hints in the readme would be helpful.) So it would be good if someone else could also check if everything works. Let me know what you think about my drawing – I sometimes found it difficult to figure out what are intentional ornaments and what is noise. I am particularly unsure about the small “hook” at the upper right part of the M letter. It looks intentional to me in the original drawing and is also present in another copy of the same initial, but looks kind of uncommon to me. M_10644 M_initial M_F1 M_F2

astoeriko avatar Jan 09 '21 18:01 astoeriko

Great! Thank you so much!

Edit: I updated #115

A couple things:

  • if you look at the original A/F/L/T, it looks like the top and the bottom of the letters are aligned on horizontal lines (green lines below). It seems all the lines should be straight as well.
  • the blue part seems to square.
  • inside the red square, I think something is missing on the right side to make it symmetric. We can barely see it in the original, but I think it's there, I'd add it.
  • I agree the hook is weird. I would remove it to make it similar to the N.
  • Can you clean your SVGs? <defs/>, <<sodipodi:namedview/>, <metadata/>, as well as the inkscape:*, sodipodi:* and style attributes should disappear. This will make sure your SVG is minimal and therefore more "standard".

M

lutetiensis avatar Jan 09 '21 19:01 lutetiensis

I was able to build the otf files but not the rest of the font files. I think the problem is that I do not have all the required software installed to run all of the build script successfully. (I had a hard time finding out what is needed, maybe some hints in the readme would be helpful.) So it would be good if someone else could also check if everything works.

You will need ttfautohint and sfnttool.jar. The latter is used to build woff and eot files, I am sure we can replace it with newer alternatives (fontforge can already generate woff files).

EDIT: with #121, you will only need ttfautohint.

I have everything working on my setup and tested your branch, everything works fine.

lutetiensis avatar Jan 10 '21 13:01 lutetiensis

Thanks for your feedback, @lutetiensis! I addressed the issues you brought up in my recent commit. The updated letter now looks like this. M

astoeriko avatar Jan 10 '21 17:01 astoeriko

Much better, thank you very much!

  1. The right half of the M doesn't seem straight. You can probably drop the anchor points along the paths.
  2. There are still some stroke-width attributes.

Otherwise I think it's ready to go in! Thanks again!

lutetiensis avatar Jan 10 '21 21:01 lutetiensis

Looks great, let's merge it 😄

doctorlard avatar May 25 '21 08:05 doctorlard