Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

Switch to FiraCode for syntax font #251

Merged
merged 1 commit into from
Aug 13, 2018
Merged

Switch to FiraCode for syntax font #251

merged 1 commit into from
Aug 13, 2018

Conversation

jeroen
Copy link
Collaborator

@jeroen jeroen commented Aug 12, 2018

Related to #246 I don't really like our current code font. It looks fancy but I find it difficult to read. For example compare some code with the current font:

old

Here the ligature joining of the fi inside the word file make it difficult to read. This is a known problem and there is a PR with a fix on the "open source" Google repo but it has been completely ignored by Google for over 2 years now: googlefonts/spacemono#2

What about using FiraCode? It is 100% open source and has ligatures including for <-, but is a bit more readable.

This is what it looks like in FiraCode. Besides being better designed, FiraCode itself is maintained by a nice person who responds to issues and pull requests.

new

@jeroen jeroen requested a review from karthik August 12, 2018 22:20
@karthik
Copy link
Contributor

karthik commented Aug 13, 2018

Looks good to me to switch. PR good to merge from my side.

@jeroen jeroen merged commit 22b8559 into master Aug 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants