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

Cropped icon rendering with Powerline+Font Awesome font . 'Fix' with opening split panes. #1769

Closed
2 tasks done
paulklemm opened this issue Apr 27, 2017 · 31 comments
Closed
2 tasks done
Labels
👆 Is Upstream Issue is the fault of something Hyper uses and not Hyper itself

Comments

@paulklemm
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

Issue

I'm using ZSH with the powerlevel9k theme and SourceCodePro+Powerline+Awesome Regular font (https://github.com/gabrielelana/awesome-terminal-fonts).

The icons are cropped in hyper.
image

When I open a new split pane horizontally, the problem disappears, while the new pane is still rendered wrong.

image

Opening a third one doesn't help.

image

If you, however, split horizontally, you can also get the opposing pane fixed.

image

@fsegouin
Copy link

Same issue here, noticed it on the previous version as well.

@paulklemm
Copy link
Author

Update: Still relevant for 1.3.3.1754

@fsegouin
Copy link

Any update on this?

@fsegouin
Copy link

If it helps: I am using fish with the oh-my-fish/theme-bobthefish theme, as well as MesloLGS Nerd Font as hyper font.

@sjoeboo
Copy link

sjoeboo commented Jun 29, 2017

+1 also having the same 'icons cut off' issue. Using Zsh w/ InputMono Nerd Font. Also seeing a split fixing it in the original window.

@tomsturge
Copy link

I've now encountered this issue. ZSH, PowerLevel9K with FureCode Nerd Font

@paulklemm
Copy link
Author

Update:

Related issues I could find:

It seems that PR #1536 solves the problem. It was merged into master end of May and should therefore be in the next Hyper.js release.

🤞🏽

@deiga
Copy link

deiga commented Jul 10, 2017

@paulklemm AFAIK #1536 has been reverted in #1847

@paulklemm
Copy link
Author

@deiga You're right. Just compiled the current codebase and the problem still persists.

@lucasqueiroz
Copy link

The interesting thing is that it only happens when there is a background color.
With colored background:
background
With default color background:
no-background

@paulklemm
Copy link
Author

@lucasqueiroz For me it's independent of the background color. Can you share your config?

@fsegouin
Copy link

@lucasqueiroz The bug you mentioning has something to do with your theme, nothing is cropped out.

@lucasqueiroz
Copy link

It's not the theme. I've removed the theme, added again. It only works when there is no background color for the text.

@tomsturge
Copy link

tomsturge commented Aug 2, 2017

Switched to iTerm as a result of this issue and it's working fine. Problem persists in Hyper with same config

Top: iTerm2
Bottom: Hyper

@tomsturge
Copy link

tomsturge commented Sep 9, 2017

Any progress on this issue? Still present in v1.4.3

@zfwf
Copy link
Contributor

zfwf commented Sep 13, 2017

Seems like vscode terminal also has the same issue: microsoft/vscode#30289. Potentially issue upstream in xterm.js?

@paulklemm
Copy link
Author

@chhschou is right, VS Code has the same problem:

screen shot 2017-09-14 at 08 56 34

@Tabrizian
Copy link

Any improvement on this issue?

@jobarbo
Copy link

jobarbo commented Sep 29, 2017

mine works in vscode but have the same issue in Hyper
capture d ecran 2017-09-29 a 12 02 05
capture d ecran 2017-09-29 a 12 15 54

@lucasqueiroz
Copy link

For me, it has been fixed in 2.x

@jobarbo
Copy link

jobarbo commented Sep 29, 2017

2.x ? isn't the last version 1.4.8?

@Tabrizian
Copy link

Tabrizian commented Sep 29, 2017 via email

@zfwf
Copy link
Contributor

zfwf commented Sep 30, 2017

On Hyper v2.0.3, still not fixed:
image
However it did seem to have improved left prompt:
image

@zfwf
Copy link
Contributor

zfwf commented Oct 13, 2017

I believe this is a simple fix, we should be using the xterm-wide-char to display the powerline characters css class instead of xterm-normal-char. Changing to xterm-wide-char in devtools looks like this:
image

I believe this is fixed upstream in xterm.js. I don't know how is xterm.js changes applied in hyper to verify this theory atm.

@fsegouin
Copy link

Seems to be fixed as of 2.0.4 (might be related to the recent fix in xterm.js).

@paulklemm
Copy link
Author

@fsegouin It's partially fixed. I still experience problems as @chhschou describes. Some Icons are still cropped.

screen shot 2017-10-23 at 12 25 36

@gabrieldrs
Copy link

gabrieldrs commented Jan 4, 2018

Hey, 2.0.4 didn't fix it for me, what did work for me though was adding this to .hyper.js config file:

termCSS: 'x-row .unicode-node {display:inline}'

@PabiGamito
Copy link

@gabrieldrs Doing this seems to mess with the cursor, slightly offsetting it to the left when there are icons on the same line. Do you happen to know of any other fix for this?

@Stanzilla
Copy link
Contributor

Hey there, to verify if this is indeed a Hyper issue or upstream at https://github.com/xtermjs/xterm.js/ could you please test if another xterm.js powered terminal shows the same problem you are experiencing? I suggest using https://github.com/Microsoft/vscode for testing. Thanks!

@PabiGamito
Copy link

@Stanzilla I was indeed experiencing the same issues in the vscode terminal. I have removed any icons to avoid this, so I haven't tested it since a bit after posting here.

@ghost
Copy link

ghost commented Apr 27, 2018

Special indicators (like the branch icon and less noticeable the git dot) are cropped in 2.1.0-canary.1:

Surprisingly this is bugged in VSCode as well.

@Stanzilla Stanzilla added the 👆 Is Upstream Issue is the fault of something Hyper uses and not Hyper itself label Apr 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👆 Is Upstream Issue is the fault of something Hyper uses and not Hyper itself
Projects
None yet
Development

No branches or pull requests