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
Add font viewing section and option to rifle #1443
Conversation
Adds fontforge as a default option, but would benefit from adding other common options for previewing fonts.
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'm actually ok with merging this with the convert because fixing the code is probably a lot of work and other convert calls might be similarly unnecessary after the fix.
So let's just open an issue on that and point to all the uses of convert in scope.sh
that would be redundant.
One small thing though, would adding some spaces to the strings create some margin at the left side of the previews? The characters starting at the very edge isn't nice imo.
- Make font-name bigger than the body - Vertically center the body so that the newline at the bottom doesn’t stand out as much
One un-double-quoted variable substitution. Directly use the commands escape code rather than relying on "$?", this also drops the need to rely on `>` which is for string comparison specifically.
Since we’re now checking whether fontimage runs successfully rather than if it throws an error, the conditions needs to be inverted.
🎉 |
ISSUE TYPE
RUNTIME ENVIRONMENT
CHECKLIST
CONTRIBUTING
document has been read [REQUIRED]DESCRIPTION
Adds fontforge as a default option for previewing fonts, but would benefit from adding other common options for previewing fonts.
MOTIVATION AND CONTEXT
There wasn't any option to preview fonts before.