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

Replace Source Code Pro with Anonymous Pro font #10826

Merged
merged 1 commit into from
Feb 1, 2020

Conversation

ZacBlanco
Copy link
Contributor

@ZacBlanco ZacBlanco commented Jan 31, 2020

The reason I'm proposing replacing SCP with AP is because there is a
bug in firefox and/or MacOS which causes this font to render
incorrectly.

See:

Anonymous Pro is a bit more lightweight and has at least been recently
updated by its maintainers (1.44.0 was 3 months ago). SCP is definitely
more ubiquitous and well-maintained but I would rather have a font that
works everywhere. As an added bonus, this font is much more
lightweight than SCP.

Additionally, it seems the maintainers of the project have no desire to
address or look into what triggered the regression:
adobe-fonts/source-code-pro#217

The reason I'm proposing replacing SCP with AP is because there is a
bug in  firefox and/or MacOS which causes this  font to render
incorrectly.

See:
- https://bugzilla.mozilla.org/show_bug.cgi?id=1534503
- https://bugzilla.mozilla.org/show_bug.cgi?id=1520157

Anonymous Pro is a bit more lightweight and has at least  been recently
updated  by its maintainers (1.44.0 was 3 months ago). SCP is definitely
more ubiquitous  and well-maintained but I would rather have a font that
works everywhere. As an added bonus, this font is *much* more
lightweight than SCP.
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • AmplabJenkins build check: PENDING
    • We were not able to detect AmplabJenkins test results on this PR. Status will update when testing completes.
  • Commits associated with Github account: PASS
  • PR title follows the conventions: PASS

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

@ZacBlanco
Copy link
Contributor Author

Here's a source with some samples of the font: https://www.marksimonson.com/fonts/view/anonymous-pro

@ZacBlanco
Copy link
Contributor Author

I also wanted to use this font, but there doesn't seem to be a well-maintained package on the npm registry: https://typeof.net/Iosevka/

Copy link
Contributor

@tieujason330 tieujason330 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, i don't think anyone will complain about the switch

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7872/
Test FAILed.

@calvinjia
Copy link
Contributor

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 5b001c9 into Alluxio:master Feb 1, 2020
@ZacBlanco ZacBlanco deleted the remove-scp branch February 3, 2020 06:36
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.

5 participants