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

Font has unwanted ligature. #730

Closed
FremyCompany opened this Issue Feb 26, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@FremyCompany

FremyCompany commented Feb 26, 2016

Initially filed against Firefox; but reproduces on Chrome too.
https://jsfiddle.net/6zqzk9fx/1/

@zarino

This comment has been minimized.

Show comment
Hide comment
@zarino

zarino Mar 11, 2016

I came here to report the same thing. I'm seeing it in Chrome on OS X.

Three examples of the 'fl' ligature which only occupies a single character width

It looks like .CodeMirror has a font-stack of:

font-family: "Fira Mono", Monaco, "Andale Mono", "Lucida Console", "Bitstream Vera Sans Mono", "Courier New", Courier, monospace;

The single-character "fl" ligature is coming from Fira Mono. If you disable Fira Mono, you get normal spacing back again. Personally I have no idea why you would want a monospace font with ligatures like this, since the whole point of a monospace font is for each character to align to a single-character grid.

zarino commented Mar 11, 2016

I came here to report the same thing. I'm seeing it in Chrome on OS X.

Three examples of the 'fl' ligature which only occupies a single character width

It looks like .CodeMirror has a font-stack of:

font-family: "Fira Mono", Monaco, "Andale Mono", "Lucida Console", "Bitstream Vera Sans Mono", "Courier New", Courier, monospace;

The single-character "fl" ligature is coming from Fira Mono. If you disable Fira Mono, you get normal spacing back again. Personally I have no idea why you would want a monospace font with ligatures like this, since the whole point of a monospace font is for each character to align to a single-character grid.

@oskarkrawczyk oskarkrawczyk self-assigned this Mar 17, 2016

@oskarkrawczyk

This comment has been minimized.

Show comment
Hide comment
@oskarkrawczyk

oskarkrawczyk Mar 17, 2016

Member

Moved to Mozilla's CDN for Fira. Should solve the problem for those who are experiencing it.

Member

oskarkrawczyk commented Mar 17, 2016

Moved to Mozilla's CDN for Fira. Should solve the problem for those who are experiencing it.

@oskarkrawczyk

This comment has been minimized.

Show comment
Hide comment
@oskarkrawczyk

oskarkrawczyk Mar 17, 2016

Member

Also fixes #720

Member

oskarkrawczyk commented Mar 17, 2016

Also fixes #720

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment