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

Remove SVG table to enable color-coded text on macOS Big Sur #125

Merged
merged 2 commits into from
Dec 22, 2020

Conversation

pkq
Copy link

@pkq pkq commented Dec 17, 2020

Remove the (experimental) SVG table to enable color-coded text on macOS Big Sur (Issue #123), following the approach implemented in upstream Source Code Pro (adobe-fonts/source-code-pro#255). See upstream issue (adobe-fonts/source-code-pro#250) for additional details.

Once merged, changes will require a new build and corresponding updates to README.md, etc.

Make DesignSpace files compliant with current specification (https://github.com/fonttools/fonttools/tree/master/Doc/source/designspaceLib) and resolve the 'No axes defined' error when building the font from source. axisDescriptors copied from upstream Source Code Pro (https://github.com/adobe-fonts/source-code-pro/blob/master/Roman/Masters/SourceCodePro.designspace)

Issue i-tu#114
Remove the (experimental) SVG table to enable color-coded text on macOS Big Sur (Issue i-tu#123), following the approach implemented in upstream Source Code Pro (adobe-fonts/source-code-pro#255). See upstream issue (adobe-fonts/source-code-pro#250) for additional details.

Refactor build.sh to be more robust, consistent with revisions to upstream code
@pkq pkq mentioned this pull request Dec 17, 2020
@i-tu
Copy link
Owner

i-tu commented Dec 22, 2020

Hey @pkq , HUGE thanks for working on this PR. I checked it out and got inspired to start working on this myself too. I don't think I would have gotten around to it if it weren't for your enthusiasm!

I have rebased the entire repository on top of the latest Source Code Pro repository, incorporating all the progress made there. These changes are now in a new default branch main. This includes the changes in this PR, but I will merge this PR into master anyway :)

@i-tu i-tu merged commit c5f8592 into i-tu:master Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants