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

[BUG] NerdFonts v3 broke the mdi-test-tube icon for neotest in nvim-notify #370

Closed
h0adp0re opened this issue Feb 20, 2024 · 4 comments
Closed
Assignees

Comments

@h0adp0re
Copy link

Describe the bug
When a Neotest notification appears via nvim-notify, the icon is an Arabic symbol instead of the old test-tube.
Screenshot 2024-02-21 at 01 20 14
This line needs to be updated:

icon = "",

To Reproduce
Use Neotest with nerd-fonts v3.

More Info
ryanoasis/nerd-fonts#1190

Possible Fix
https://github.com/loichyan/nerdfix

@h0adp0re
Copy link
Author

And offtopic — for some reason I got an email immediately after filing this issue. Is this a GH bug?

Add issues to tracking project: All jobs have failed
workflow run failed in 4 seconds - master (f6048f3)

@Finii
Copy link

Finii commented Mar 30, 2024

FB67 had to be removed because the Nerd Font icons in that plane broke a lot international fonts. That codepoint is not 'free to use' (i.e. not PUA).
For that reason all MD icons got moved with Nerd Fonts v3.

image

rcarriga added a commit that referenced this issue Apr 8, 2024
@rcarriga
Copy link
Collaborator

rcarriga commented Apr 8, 2024

Switched to using an icon from codicons to be consistent and made it configurable

@rcarriga rcarriga closed this as completed Apr 8, 2024
@rcarriga
Copy link
Collaborator

rcarriga commented Apr 8, 2024

And offtopic — for some reason I got an email immediately after filing this issue. Is this a GH bug?

I've no idea why you got that, it must be 😅

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

3 participants