Skip to content

Commit

Permalink
Rolled back ascender/descender changes from 6ee86a6 #798
Browse files Browse the repository at this point in the history
  • Loading branch information
tonsky committed Aug 1, 2019
1 parent cf056b2 commit c13a803
Show file tree
Hide file tree
Showing 29 changed files with 45 additions and 14 deletions.
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,5 @@ FiraCode\ (Autosaved).glyphs
master_ufo
instance_ufo
venv
.DS_Store
.DS_Store
*.numbers
24 changes: 12 additions & 12 deletions FiraCode.glyphs
Original file line number Diff line number Diff line change
Expand Up @@ -236,31 +236,31 @@ capHeight = 1374;
customParameters = (
{
name = typoAscender;
value = 1980;
value = 1870;
},
{
name = typoDescender;
value = -644;
value = -530;
},
{
name = typoLineGap;
value = 0;
},
{
name = winAscent;
value = 2164;
value = 1870;
},
{
name = winDescent;
value = 1000;
value = 530;
},
{
name = hheaAscender;
value = 1980;
value = 1870;
},
{
name = hheaDescender;
value = -644;
value = -530;
},
{
name = hheaLineGap;
Expand Down Expand Up @@ -306,31 +306,31 @@ capHeight = 1386;
customParameters = (
{
name = typoAscender;
value = 1980;
value = 1870;
},
{
name = typoDescender;
value = -644;
value = -530;
},
{
name = typoLineGap;
value = 0;
},
{
name = winAscent;
value = 2164;
value = 1870;
},
{
name = winDescent;
value = 1000;
value = 530;
},
{
name = hheaAscender;
value = 1980;
value = 1870;
},
{
name = hheaDescender;
value = -644;
value = -530;
},
{
name = hheaLineGap;
Expand Down
Binary file modified distr/otf/FiraCode-Bold.otf
Binary file not shown.
Binary file modified distr/otf/FiraCode-Light.otf
Binary file not shown.
Binary file modified distr/otf/FiraCode-Medium.otf
Binary file not shown.
Binary file modified distr/otf/FiraCode-Regular.otf
Binary file not shown.
Binary file modified distr/otf/FiraCode-Retina.otf
Binary file not shown.
Binary file modified distr/ttf/FiraCode-Bold.ttf
Binary file not shown.
Binary file modified distr/ttf/FiraCode-Light.ttf
Binary file not shown.
Binary file modified distr/ttf/FiraCode-Medium.ttf
Binary file not shown.
Binary file modified distr/ttf/FiraCode-Regular.ttf
Binary file not shown.
Binary file modified distr/ttf/FiraCode-Retina.ttf
Binary file not shown.
Binary file modified distr/variable_ttf/FiraCode-VF.ttf
Binary file not shown.
Binary file modified distr/woff/FiraCode-Bold.woff
Binary file not shown.
Binary file modified distr/woff/FiraCode-Light.woff
Binary file not shown.
Binary file modified distr/woff/FiraCode-Medium.woff
Binary file not shown.
Binary file modified distr/woff/FiraCode-Regular.woff
Binary file not shown.
Binary file modified distr/woff/FiraCode-Retina.woff
Binary file not shown.
Binary file modified distr/woff/FiraCode-VF.woff
Binary file not shown.
Binary file modified distr/woff2/FiraCode-Bold.woff2
Binary file not shown.
Binary file modified distr/woff2/FiraCode-Light.woff2
Binary file not shown.
Binary file modified distr/woff2/FiraCode-Medium.woff2
Binary file not shown.
Binary file modified distr/woff2/FiraCode-Regular.woff2
Binary file not shown.
Binary file modified distr/woff2/FiraCode-Retina.woff2
Binary file not shown.
Binary file modified distr/woff2/FiraCode-VF.woff2
Binary file not shown.
1 change: 1 addition & 0 deletions script/bootstrap
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ brew install ttfautohint
brew install woff2
brew tap bramstein/webfonttools
brew install sfnt2woff-zopfli
pip3 install fontbakery
2 changes: 1 addition & 1 deletion script/build
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash -ex
#!/bin/bash -e

source venv/bin/activate

Expand Down
24 changes: 24 additions & 0 deletions script/build_ttf
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
#!/bin/bash -e

source venv/bin/activate

This comment has been minimized.

Copy link
@fabiangreffrath

fabiangreffrath Jun 9, 2020

What's the matter with this file?

This comment has been minimized.

Copy link
@tonsky

tonsky Jun 9, 2020

Author Owner

this is where all the python libs live. It’s built by the bootstrap script

This comment has been minimized.

Copy link
@fabiangreffrath

fabiangreffrath Jun 9, 2020

Ah, okay, thanks! In Debian we rebuild the fonts using only packages from the official repository, so I will either have to patch out this line or provide an empty file with that path.

BTW, am I right that this is the preferred build script for rebuilding the fonts in Truetype file format?

This comment has been minimized.

Copy link
@tonsky

tonsky Jun 9, 2020

Author Owner

BTW, am I right that this is the preferred build script for rebuilding the fonts in Truetype file format?

yes. It’s not the same I use (I export directly from Glyphs.app), but I think it’s the one Google fonts uses


args=( "$@" )
default_weights=( "Light" "Regular" "Retina" "Medium" "Bold" )
weights=( "${args[@]:-"${default_weights[@]}"}" )

for weight in "${weights[@]}"; do
file=distr/ttf/FiraCode-${weight}.ttf

echo "Making " ${file}
rm -rf ${file}
fontmake -g FiraCode.glyphs -o ttf --output-dir distr/ttf -i "Fira Code ${weight}"

echo "Fixing DSIG in " ${file}
gftools fix-dsig --autofix ${file}

This comment has been minimized.

Copy link
@fabiangreffrath

fabiangreffrath Jun 21, 2020

What exactly is this tool and what does it do here? Is this step strictly necessary to build the font?

I am asking because we don't have this tool in Debian yet, but I'd like to rebuild the font as faithfully to the official build chain as possible.

This comment has been minimized.

Copy link
@tonsky

tonsky Jun 21, 2020

Author Owner

https://github.com/googlefonts/gftools
I’m not sure, it fixes something. It was added by Google Fonts team


echo "TTFautohint " ${file}
hintedFile=${file/".ttf"/"-hinted.ttf"}
ttfautohint -I ${file} ${hintedFile} --stem-width-mode nnn --composites
# --windows-compatibility
mv ${hintedFile} ${file}
done
5 changes: 5 additions & 0 deletions script/check
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
#!/bin/bash -e

source venv/bin/activate

fontbakery check-universal -C $1

0 comments on commit c13a803

Please sign in to comment.