Skip to content
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 Italics style #27

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add Italics style #27

wants to merge 1 commit into from

Conversation

alerque
Copy link
Member

@alerque alerque commented Aug 22, 2020

This is a rebase of @cssobral2013 #22 with the only thing that matters: the source. Unfortunately since that PR was opened from the owner's master branch I can't help directly by editing the PR.

It isn't building properly yet so let me take care of whatever that is about, then I'll comment with instructions on how to either get this moved over there or whatever needs to be done next...

@cssobral2013
Copy link

@alerque - I cannot test it now because I use Windows, so I cannot use Glyphs.

@alerque
Copy link
Member Author

alerque commented Aug 31, 2020

Thanks @cssobral2013. I'm in much the same boat being on Linux. I've tagged both your original PR and this on as help wanted and would encourage anybody who is interested in taking this on to drop a comment and I'd be glad to help work through the process. But we do need somebody with Glyphs and a working knowledge of it.

Eventually Fontship should facilitate the process of contributing to fonts using any editor by providing conversion between source formats, but that bit is under-developed right now.

Additionally this PR should be building the Italic font as ready to use binary artifacts that anybody could test, but there is a syntax problem with the Glyphs file. Until that is worked out we can't even really preview how this would fit in.

Again to anybody interested in helping, please drop a line here and I'll help you get rolling.

@alerque alerque mentioned this pull request Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants