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

Update web font to Atkinson Hyperlegible #938

Merged
merged 4 commits into from
Feb 21, 2021
Merged

Conversation

weiji14
Copy link
Member

@weiji14 weiji14 commented Feb 20, 2021

Description of proposed changes

Use Atkinson Hyperlegible Font. Preview at https://pygmt-git-docs-fonts-legible-gmt.vercel.app/gallery/line/vector-heads-tails.html#sphx-glr-gallery-line-vector-heads-tails-py. Sample:

Atkinson Hyperlegible font sample preview

Pros: Letters like small letter L has a little slant at the bottom that differentiates it from i,
Cons: The number 0 character has a slash in the middle which may (or may not) look ugly.

License appears to be modified from Open Font License, see diff at google/fonts#2816 (comment)

References:

Fixes #933

Reminders

  • Run make format and make check to make sure the code follows the style guide.
  • Add tests for new features or tests that would have caught the bug that you're fixing.
  • Add new public functions/methods/classes to doc/api/index.rst.
  • Write detailed docstrings for all functions/methods.
  • If adding new functionality, add an example to docstrings or tutorials.

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash commands are:

  • /format: automatically format and lint the code
  • /test-gmt-dev: run full tests on the latest GMT development version

@weiji14 weiji14 added the documentation Improvements or additions to documentation label Feb 20, 2021
Also dropped Open Sans, and set standard font-weight values.
Copy link
Member

@liamtoney liamtoney left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@weiji14 weiji14 changed the title Update web font (Atkinson Hyperlegible) Update web font to Atkinson Hyperlegible Feb 21, 2021
@weiji14 weiji14 marked this pull request as ready for review February 21, 2021 22:23
@weiji14 weiji14 merged commit 8e9ee3f into master Feb 21, 2021
@weiji14 weiji14 deleted the docs-fonts-legible branch February 21, 2021 22:27
@seisman seisman added this to the 0.3.1 milestone Feb 21, 2021
sixy6e pushed a commit to sixy6e/pygmt that referenced this pull request Dec 21, 2022
Use [Atkinson Hyperlegible Font](https://brailleinstitute.org/freefont)
instead of Open Sans so that letters like i and l can be differentiated.
Fonts are served using Web Open Font Format 2 (woff2) format.
Also set body and paragraph font-size to 1.05em, and set standard
font-weight values.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better web font for I (capital i) and l (small L)
3 participants