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

Certain Neotree icons cannot be displayed #180

Closed
ppwwyyxx opened this issue May 6, 2018 · 11 comments · Fixed by #726
Closed

Certain Neotree icons cannot be displayed #180

ppwwyyxx opened this issue May 6, 2018 · 11 comments · Fixed by #726

Comments

@ppwwyyxx
Copy link

ppwwyyxx commented May 6, 2018

I've install fonts from all-the-icons, and after (setq doom-neotree-file-icons t), I can see icons for most filetypes except for certain files:
0505-22 57 10

Then I modified modules/ui/doom/config.el and remove the doom-themes-neotree-config hook, the icons for Makefile and LICENSE are back, just like the official neotree screenshot: https://github.com/jaypei/emacs-neotree/blob/dev/screenshots/icons.png

So it seems like doom-themes-neotree has some issues with files without extensions.

@hlissner
Copy link
Member

hlissner commented May 6, 2018

Is there any reason you can't use (setq neo-theme 'icons) instead of doom-themes-neotree-config?

I've been meaning to phase out support for (setq doom-neotree-file-icons t), because it does little that neotree's icon theme can't. Rather, the purpose of doom-themes-neotree-config is to provide a minimalistic icon theme (ala atom), e.g.

image

This is meant as an alternative to neotree's full-blown icon theme.

@ppwwyyxx
Copy link
Author

ppwwyyxx commented May 6, 2018

(setq neo-theme 'icons) seems to have the same issue.

@hlissner
Copy link
Member

hlissner commented May 6, 2018

Even with doom-themes-neotree-config removed from your config?

@ppwwyyxx
Copy link
Author

ppwwyyxx commented May 6, 2018

When doom-themes-neotree-config was removed things work fine.
Do you mean that I should just disable it if I want icons?

@hlissner
Copy link
Member

hlissner commented May 6, 2018

Yes, if you want icons-by-filetype (like in your post), use (setq neo-theme 'icons) without doom-themes-neotree-config.

But if you want atom-style icons (like in my post above), use

(doom-themes-neotree-config)
;; this isn't necessary, because 'simple is the default
(setq doom-neotree-file-icons 'simple)

@ppwwyyxx
Copy link
Author

ppwwyyxx commented May 6, 2018

Thanks! I end up with

(def-package-hook! doom-themes
  :post-config
  (remove-hook 'doom-load-theme-hook #'doom-themes-neotree-config)
)

@ppwwyyxx ppwwyyxx closed this as completed May 6, 2018
@hlissner
Copy link
Member

hlissner commented May 6, 2018

Oh! I wasn't aware you were a Doom user. In that case, I'd recommend doing it this way:

(after! doom-themes
  (remove-hook 'doom-load-theme-hook #'doom-themes-neotree-config))

It is functionally equivalent, but after! is generally more preformant than (def-package-hook! X :post-config ...).

hlissner added a commit that referenced this issue May 7, 2018
Neotree provides this (with better support than doom-themes provides).
Use (setq neo-theme 'icons) for per-filetype file icons. Doom's neotree
theme is minimalistic, and takes after Atom's project drawer icons.
@jupl
Copy link
Contributor

jupl commented May 9, 2018

I'm pretty bummed out by this move, unfortunately after updating recently. If I don't use doom-themes-neotree-config I lose the nice customizations that were offered by doom.

screen shot 2018-05-09 at 6 58 29 am

However, when I use doom-themes-neotree-config I don't get any of the file icons that are useful at a glance, even with the neo-theme option.

screen shot 2018-05-09 at 6 57 19 am

Is there any way to get the best of both worlds like how it used to work?

@hlissner
Copy link
Member

hlissner commented May 9, 2018

@jupl Sorry about that. To remove it entirely was too hasty a decision. I'll revisit this in the next day or two and bring doom-themes-neotree-config up to snuff so I can restore that feature.

@rickithadi
Copy link

rickithadi commented Jan 16, 2019

@hlissner any updates on this?

@jesusbv
Copy link

jesusbv commented Dec 7, 2020

Thank you for the package and the efforts on this repo @hlissner

I would like to ask if it is possible to extend/add more icons ?

for example, without

(doom-themes-neotree-config)

Screenshot from 2020-12-07 11-52-21

with

(doom-themes-neotree-config)
(setq doom-themes-neotree-file-icons t)

Screenshot from 2020-12-07 11-52-42

Most of icons are there but some are missing

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 14, 2021
archiif added a commit to archiif/themes that referenced this issue May 12, 2022
Fixes an old issue where some icons are missing with 
`(setq doom-themes-neotree-file-icons t)`

Fix: doomemacs#180 (comment)
hlissner pushed a commit that referenced this issue May 26, 2022
Fixes an old issue where some icons are missing with 
`(setq doom-themes-neotree-file-icons t)`

Fix: #180 (comment)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants