mikeslab/Iosevka
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions 1

Fix the metrics of wide characters, making Extended families compatible with FontConfig’s “dual” width (#1247).

Browse source
This commit is contained in:
be5invis 2021-11-05 18:52:11 -07:00
parent 8270d53730
commit 81827f16ba
15 changed files with 14 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

9
font-src/gen/finalize/index.js
View file

@ -31,11 +31,16 @@ function assignSubRank(glyphStore) {
// (AW > 0) have the same width. We use this method to validate whether our "Fixed" subfamilies
// are properly built.
function validateMonospace(para, glyphStore) {
if (!para.forceMonospace) return;
let awSet = new Set();
for (const [u, g] of glyphStore.encodedEntries()) {
const aw = Math.round(g.advanceWidth || 0);
if (aw > 0) awSet.add(aw);
}
if (awSet.size > 1) throw new Error("Unreachable! Fixed variant has wide characters");
if (para.forceMonospace && awSet.size > 1) {
throw new Error("Unreachable! Fixed variant has wide characters");
}
if (!para.isQuasiProportional && awSet.size > 2) {
console.log(awSet);
throw new Error("Unreachable! Fixed variant has wide characters");
}
}