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

Disable font ligatures #11631

Closed
wants to merge 1 commit into from
Closed

Conversation

silverwind
Copy link
Member

Some fonts like Roboto autmatically form ligatures but I think they make text harder to read and should be avoided.

Before:

image

After:

image

@zeripath
Copy link
Contributor

heh so this is where I would have to disagree. I much prefer ligatures in text.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 26, 2020
@silverwind
Copy link
Member Author

silverwind commented May 26, 2020

I guess this will be a controversial one. I hate them 😉.

That 'fl' could be mistaken for a A for people with bad sight. Also, I think I've never seen a ligature on github.com but I guess that's because system fonts generally don't ligature.

@lafriks
Copy link
Member

lafriks commented May 26, 2020

@silverwind for me in github actually in your comment fl are shown with ligature :)

@lafriks lafriks added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label May 26, 2020
@CirnoT
Copy link
Contributor

CirnoT commented May 26, 2020

No ligature on GitHub here; are you using macOS by any chance?

@lafriks
Copy link
Member

lafriks commented May 26, 2020

Android :)

@silverwind
Copy link
Member Author

Yeah, I think Roboto is the only system-level font that forms ligatures by default.

BTW, I'm now using a userstyle to disable ligatures on all sites 😉

@silverwind
Copy link
Member Author

Doesn't look like this will gain enough acceptance.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants