-
Notifications
You must be signed in to change notification settings - Fork 219
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
Fixes getting path from woff files with CFF #200
Conversation
test/glyphs.js
Outdated
return assert.equal(glyph.name, 'T'); | ||
}); | ||
describe('WOFF otf glyphs', function() { | ||
let font = fontkit.openSync(__dirname + '/data/SourceSansPro/SourceSansPro-Regular.otf.woff'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think there's a prettifier running for the source at the moment, but it looks like this got indented twice.
test/glyphs.js
Outdated
describe('WOFF2 glyph', function() { | ||
let font = fontkit.openSync(__dirname + '/data/SourceSansPro/SourceSansPro-Regular.woff2'); | ||
describe('WOFF2 ttf glyph', function() { | ||
let font = fontkit.openSync(__dirname + '/data/SourceSansPro/SourceSansPro-Regular.ttf.woff2'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like there's double indentation here, too.
test/glyphs.js
Outdated
}); | ||
|
||
describe('WOFF2 otf glyph', function() { | ||
let font = fontkit.openSync(__dirname + '/data/SourceSansPro/SourceSansPro-Regular.otf.woff2'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like there's double indentation here, too
03a31d5
to
9a9f0c2
Compare
Fixes #199
Uses stream from CFF table instead of font one which does not match in woff fonts