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

Fira Code ligatures not showing #914

Closed
ericmarkmartin opened this issue Oct 21, 2016 · 30 comments
Closed

Fira Code ligatures not showing #914

ericmarkmartin opened this issue Oct 21, 2016 · 30 comments

Comments

@ericmarkmartin
Copy link

ericmarkmartin commented Oct 21, 2016

After updating Hyperterm to 0.8.3 through homebrew cask, the Fira Code font's ligatures don't work, even though the font itself is being used
screen shot 2016-10-20 at 11 49 09 pm

In the above screenshot, the 0 especially gives away that FiraCode is being used and not Menlo.

screen shot 2016-10-20 at 11 51 50 pm

Yet as you can see from the arrow function above, the ligatures are not being used.

@0x6c7862
Copy link

This CSS will re-enable ligatures:

x-screen x-row { font-variant-ligatures: contextual; }

@wolverian
Copy link

FWIW, even if I add the CSS to Hyper's config (the css key), the x-row elements don't get that style. 🤔

@dak0rn
Copy link

dak0rn commented Oct 21, 2016

As suggested in another issue, I added the following to the configuration to make ligatures work. I am using Hasklig instead of Fira Code, though, but I expect the font not to be the issue.

screen shot 2016-10-21 at 12 31 23

@iamstarkov
Copy link
Contributor

definitely a bug, but also a dup #874

@ericmarkmartin
Copy link
Author

Thanks for the replies. Unfortunately I didn't see that issue since because it was marked closed, not a bug, and didn't contain the word Fira.

@mjswensen
Copy link

Thanks for the workaround, @0x6c7862 and @dak0rn ! Works great for now.

I didn't see any code in this repo explicitly turning them off so my guess is that a dependency upstream somewhere changed and broke this behavior.

I'd vote for ligatures being enabled by default unless there's a good reason not to. People coming to a terminal based on JS, HTML, and CSS will expect ligatures to work out of the box.

@L-A L-A mentioned this issue Dec 1, 2016
2 tasks
@ccorcos
Copy link

ccorcos commented Dec 1, 2016

Hmm. I wasn't able to get it to work:

image

image

@fczuardi
Copy link

fczuardi commented Dec 2, 2016

@ccorcos the relevant lines:

    // font family with optional fallbacks
    fontFamily: '"Fira Code", Menlo, "DejaVu Sans Mono", "Lucida Console", monospace',

and

    // custom css to embed in the terminal window
    termCSS: `
        x-screen x-row {
            font-variant-ligatures: initial;
        }
    `,

@ccorcos
Copy link

ccorcos commented Dec 2, 2016

wait, now its working! :)

@gluons
Copy link

gluons commented Feb 21, 2017

Is it posible to add a simple boolean option to trigger this feature?

@thasmo
Copy link

thasmo commented Feb 25, 2017

enableLigatures pls :D

@matheuss
Copy link
Member

It is @gluons

@Qix-
Copy link

Qix- commented May 12, 2017

Just curious; why are ligatures disabled by default? Is there an inherent overhead?

@timneutkens
Copy link
Member

Just published https://github.com/timneutkens/hyper-ligatures. Guess we can close the issue 👍

@fleischie
Copy link

@timneutkens Just fyi (you probably already saw, but for completeness sake). As Hyper v2 uses a different terminal emulation, that is using different tags/classNames as rows, there should be a change to your plugin to accommodate for this. (I created a PR on your repo with the changes I think are necessary to support the new implementation as well).

😄

@albinekb
Copy link
Contributor

@timneutkens this will break in future updates of hyper. (More exactly, when we change to xterm 3, based on canvas) you should state that in your readme 👍

@timneutkens
Copy link
Member

@albinekb by then lets add ligatures built in ;p

@SanketDG
Copy link

Does the above solution work with the very latest pre release version?

@albinekb
Copy link
Contributor

Nope @SanketDG hyper won't have support for ligatures until xterm3+ does 👍

@iamstarkov
Copy link
Contributor

=( i will miss ligatures
screen shot 2018-01-18 at 9 49 59 pm

@caiocutrim
Copy link

no working for me in Ubuntu and hyper 1.4.8

@Xunnamius
Copy link

This is incredibly sad, but at least they show up in Sublime so I'm not too disappointed 😄 (confirmed not working as of 2.0.0-11)

@Qix-
Copy link

Qix- commented Jan 28, 2018

Ref xtermjs/xterm.js#958

@JCMais
Copy link

JCMais commented Aug 30, 2018

addon for xterm with ligatures here https://github.com/xtermjs/xterm-addon-ligatures (from above issue)

@pokatomnik
Copy link

Win 10, the latest hyper.app build (I have just updated It from the stable channel), ligatures does not appear (i.e. => does not show as an arrow)

@williamleiby
Copy link

latest hyper update on macOS Mojave, font ligatures don't work. Would like a useful way of implementing them.

@Neo42
Copy link

Neo42 commented Dec 18, 2018

#2793

@williamleiby Try the method for this issue. Worked perfectly for me.

@aadibajpai
Copy link

Win 10, the latest hyper.app build (I have just updated It from the stable channel), ligatures does not appear (i.e. => does not show as an arrow)

did you find a fix?

@dankolesnikov
Copy link

Any updates on this? Just discovered Fira Code and really want to get it working on Hyper.

@bet4it
Copy link
Contributor

bet4it commented Dec 8, 2019

@dankolesnikov #3607

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

No branches or pull requests