Merge pull request #137 from ramirezcgn/master

Fix normalize option with wide icons
nfroidure · Sep 19, 2021 · be4d9ba · be4d9ba
2 parents eb26c81 + 54dd0e6
commit be4d9ba
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/index.js b/src/index.js
@@ -433,7 +433,7 @@ class SVGIcons2SVGFontStream extends Transform {
 
     this.glyphs.forEach((glyph) => {
       const ratio = this._options.normalize
-        ? fontHeight / glyph.height
+        ? fontHeight / (glyph.width > glyph.height ? glyph.width : glyph.height)
         : fontHeight / maxGlyphHeight;
       if (!isFinite(ratio)) throw new Error('foo');
       glyph.width *= ratio;

diff --git a/tests/expected/variableheighticonsn.svg b/tests/expected/variableheighticonsn.svg
diff --git a/tests/results/variableheighticonsn.svg b/tests/results/variableheighticonsn.svg