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

Weird characters showing since upgrade to 3.1.0 #5757

Closed
2 tasks done
corneliusroemer opened this issue Jul 17, 2021 · 14 comments
Closed
2 tasks done

Weird characters showing since upgrade to 3.1.0 #5757

corneliusroemer opened this issue Jul 17, 2021 · 14 comments

Comments

@corneliusroemer
Copy link

  • I am on the latest Hyper.app version
  • I have searched the issues of this repo and believe that this is not a duplicate
  • OS version and name: Ubuntu 20.04
  • Hyper.app version: 3.1.0
  • Link of a Gist with the contents of your .hyper.js: Vanilla
  • The issue is reproducible in vanilla Hyper.app: Yes

Issue

I updated and have some very weird characters appearing.

I use fish shell and fira code nf.

I downgraded to 3.0.2 again and it's fine now.

image

@corneliusroemer
Copy link
Author

Other people seem to have the same problem: #5753 (comment)

@LabhanshAgrawal
Copy link
Collaborator

#5753 (reply in thread) mentions that removing ligatures plugin solved it.
You've written that you are using vanilla config, can you confirm again?

@corneliusroemer
Copy link
Author

This is my config, the only change is that I now use "Fira Code NF" as font (which has ligatures). No plugin enabled.

https://gist.github.com/corneliusroemer/9783395f7793aa8b79ea6c08a7dd6380

@pregnor
Copy link

pregnor commented Jul 18, 2021

I experience the same on macOS 11.4 (Big Sur, amd), Hyper 3.1.0, bash shell with Fira Code font (contains ligatures), .hyper.js [removed after resolution].

I don't have the font-ligatures plugin, but I do have other plugins installed, although IMO none of those should interfere with the shell display area.

@LabhanshAgrawal
Copy link
Collaborator

Looks like xterm ligatures plugin is not playing well with webgl renderer.
Can you all try setting

disableLigatures: true

and check if it's fixed.

P.S. If you want to use ligatures, you will have to use canvas renderer by setting webGLRenderer: false

@pregnor
Copy link

pregnor commented Jul 19, 2021

For me using either disableLigatures: true or webGLRenderer: false successfully fixes the issue. 👍

(Also not sure if it helps with anything, but the messed up text seems to be deterministic. I can reliably issue an ls -al command in my home directory and always have the same characters messed up when both ligatures and webGLRenderer are turned on.)

@LabhanshAgrawal
Copy link
Collaborator

I'll disable webgl and ligatures combination until it's fixed in xterm then.

@corneliusroemer
Copy link
Author

Great, where/when can I test?

@LabhanshAgrawal
Copy link
Collaborator

Doing a patch release soon.

@corneliusroemer
Copy link
Author

Just tested 3.1.1, looks good so far. Will close for now.

@gerdm
Copy link

gerdm commented Jul 19, 2021

Hi, I just downloaded version 3.1.1 and I still seem to have the same problem. Any suggestions?
Screenshot 2021-07-19 at 13 30 59

Screenshot 2021-07-19 at 13 30 47

I've also tried to set webGLRenderer: false, and disableLigatures: true

@corneliusroemer
Copy link
Author

Ok, I reopen because of @gerdm's report. For me, it is fixed though (at least haven't seen any problems yet).

@LabhanshAgrawal
Copy link
Collaborator

@gerdm The about dialog is showing webgl as the renderer
you need to open a new window after disabling webgl
also you are using a plugin for ligatures which would enable the ligatures addon despite disabling it in the config.

@gerdm
Copy link

gerdm commented Jul 19, 2021

Hi @LabhanshAgrawal,

You're correct, I had a plugin that was using ligatures. Removing that plugin made everything work fine.
Thank you!

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

4 participants