SamsaGlyph.prototype.svg has hardcoded width and height

[wentin]

Some fonts are not based on 1000 units, it would be better if the width and height of the SVG can be set dynamically, based on the font data, such as unitsPerEm
Right now it is being hardcoded as width="1000" height="1000"

SamsaGlyph.prototype.svg = function (style={}) {
	...
	return `<svg version="1.1" xmlns="http://www.w3.org/2000/svg" width="1000" height="1000">
	<g${style.transform ? ` transform="${style.transform}"` : ""}>
		<path d="${this.svgPath()}"${extra}></path>
	</g>
</svg>`;
}

[wentin]

#75
I submitted a Pull Request to solve this issue.
Please note: I think it would be better to set the svg width based on this glyph's actual canvas width, but don't know what is the best way to access it. I settled for unitsPerEm for now.

[wentin]

alternative pull request #76
use viewBox instead of width and height, this will make the SVG responsive

[Lorp]

Thanks for this! I’ve been meaning to improve the SVG sizing for a while. I’ll take a look soon and almost certainly accept your PR.