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

fix: replace emoji for CMake and Erlang with NerdFonts #1787

Merged
merged 1 commit into from
Oct 13, 2020

Conversation

matchai
Copy link
Member

@matchai matchai commented Oct 13, 2020

Description

The emoji used for the CMake and Erlang modules appear to have little font support, as is shown on the emojipedia entries for the respective emoji:
https://emojipedia.org/triangle-with-rounded-corners/
https://emojipedia.org/three-networked-computers/

This PR replaces the emoji used with NerdFont icons for "Triangle Outline" and "Erlang".

Motivation and Context

Closes #

Screenshots (if appropriate):

How Has This Been Tested?

  • I have tested using MacOS
  • I have tested using Linux
  • I have tested using Windows

Checklist:

  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.

@matchai matchai changed the title fix: replace emoji for cmake and erlang fix: replace emoji for CMake and Erlang with NerdFonts Oct 13, 2020
@ryanpcmcquen
Copy link

Thank you @matchai!

@matchai matchai merged commit 783eb33 into master Oct 13, 2020
@matchai matchai deleted the replace-emoji-with-nerdfonts branch October 13, 2020 22:01
@wilbeibi
Copy link

Ha, this is why my cpp project show chinese character "喝" ("drink") in prompt.

@DCjanus
Copy link

DCjanus commented Oct 27, 2020

Hi, "喝" in Chinese means "drink", "喝" as CMake project symbol is really weird to me, a Chinese native speaker.

@matchai
Copy link
Member Author

matchai commented Oct 27, 2020

The character occupies the same Unicode codepoint as the symbol we are aiming to use in the Nerd Fonts version of fonts.

If you are seeing the Chinese character for "drink" try changing your current font to a Nerd Font patched version.

@mattn
Copy link

mattn commented Nov 10, 2020

This PR break the prompt for Japanese too. I think this have better to be reverted.

image

@matchai
Copy link
Member Author

matchai commented Dec 18, 2020

@mattn, @DCjanus
It looks like the Nerd Fonts collision is caused by ryanoasis/nerd-fonts#365.
We will be updating the prompt to use a new symbol until this issue is closed. 👍

chipbuster pushed a commit to chipbuster/starship that referenced this pull request Jan 14, 2021
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

Successfully merging this pull request may close these issues.

5 participants