-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Geom #4874
Comments
For some reason the “fonts” folder is in the “sources” folder. That’s not quite right, it should be in the main directory. (Both to avoid confusion and for submission to Google.) Also, the Regular weight is named “Rergular” for both the OTF and TTF files. |
@tphinney Thanks for the input. I changed the location of the "fonts" folder. |
That gets complicated, but I think Google’s standard is to include the style name in there. (Technically the OpenType spec does not require it, and some Adobe fonts have omitted it in the past, but I for one have always thought that was a bit odd. IIRC, they mostly only did that in their earliest fonts up to around 1990 or so, and mostly when the font was a single-member family.) I was just pointing out that “Regular” was spelled incorrectly, with an extra “r” |
Now I get it. I never saw the extra "r". Thanks! |
@RosaWagner What exactly does the label "Needs design work" ? |
More of what was previously described here https://github.com/ThanosPoulakidas/Geom/tree/main/sources has more than one glyphs source file which is strange, and the style names in the extraneous file names don't match our requirements. |
I think it should be ok now? |
Each time an error in the Build Font Phase keeps appearing. 186 |
I should leave this to the folks who know GF Builder, which I don’t use yet. So that suggests a problem in the Glyphs file, or at least in GF Builder being able to parse it. Probably need to post (or point to) that Glyphs file for help. |
I checked the Glyphs and there was a problem with 7 glyphs during the export on the variable format and I corrected it but the same problem appears again. |
Hi, |
@felipesanches I suggest you could schedule a time for a call with @ThanosPoulakidas to better understand the need of users here |
@felipesanches I wiil be waiting for your invitation |
The problem I am facing this moment, is there a possibility to be similar to this? |
@simoncozens I have just reuplaoded the files |
Great, thank you! Yes, the problem here is that you are using quadratic curves and the overlap removal step is failing. You could convert all your curves to cubic using the Path > Other > Convert to Cubic menu option in glyphs, or if you really want to design with quadratics, you can add the item:
in your |
@simoncozens removeOutlineOverlaps: false But does the way I placed it makes a difference? |
Let me work on it and send you a PR. |
How do I install the latest FontBakery? |
Install info is at: https://font-bakery.readthedocs.io/en/latest/ You can also run it online via a web site at http://fontbakery.com/ — it runs locally in your web browser and does not upload the font(s) or anything. (BUT the web browser version can’t run all the possible tests. It does do most of them, however!) |
First of all, thanks for the links. |
Do I have to update the fontbakery? Because I am checking other repositories which they also have older versions of the fontbakery |
Is there something else that I need to work on? |
Font Project Git Repo URL:
https://github.com/ThanosPoulakidas/Geom.git
Previous Issue URL:
#4525
Super short description of the Font Family:
Geom, is a contemporary geometric sans serif typeface. It has 7 weights. This typeface is intended for display purposes but it works just as well in small texts. Even though the geometric typefaces look more dynamic in relation with other typefaces, Geom has gone one step further by having an even more additional dynamic characteristics on some of its characters. The typeface is a variable font which includes a Latin and Greek character set.
Requirements:
I understand that Google Fonts will publish only fonts that matches its requirements, and I can confirm the project meets them (by ticking the cases, or putting x between the square brackets in text mode):
Image:
Attach here a pic or a screenshot of the font; one is enough, it can be few letters, to give a quick overview.
The text was updated successfully, but these errors were encountered: