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

Mode-icons addition #5632

Closed
ReneFroger opened this issue Mar 29, 2016 · 21 comments
Closed

Mode-icons addition #5632

ReneFroger opened this issue Mar 29, 2016 · 21 comments
Labels
- Mailling list - Feature request New Package stale marked as a stale issue/pr (usually by a bot)

Comments

@ReneFroger
Copy link

Check the repo mode-icons for Emacs.

I think it's benefical for Spacemacs. It makes more distinct what the major mode is. What do you think?

@syl20bnr
Copy link
Owner

Seems a nice addition if it looks nice with resizing etc... otherwise it may be considered a user setting and be used with dotspacemacs-additional-packages.

@ReneFroger
Copy link
Author

I just tried it, and it scales well. Or perhaps do you mean else with it?

And I'm wondering why you would consider it as an user setting? It seems to me it's a nice feature. Why not consider Magit as an user setting? Or Helm? Or Spacemacs, because Spacemacs is all about custom user settings after all.

@syl20bnr
Copy link
Owner

Because we don't create layers just to wrap a package, there is the dotspacemacs-additional-packages for this.
If it looks nice (no weird and obvious graphical artefacts) we can add it to the spacemacs distribution, I'll try it to see if it fits there.

@ragloo
Copy link

ragloo commented Apr 7, 2016

it's very nice and sexy but mode-icons sadly breaks helm alignement for me

@StreakyCobra
Copy link
Contributor

@ragloo Screenshot and reproduction steps would help :-)

@syl20bnr
Copy link
Owner

syl20bnr commented Apr 7, 2016

We can disable it in helm buffers I guess.

@ReneFroger
Copy link
Author

How would you disable it for Helm then? I tried it but I couldn't figure it out.

@xcodebuild
Copy link
Contributor

@ragloo @ReneFroger (setq mode-icons-change-mode-name nil) would disable it in helm or ido etc

BTW, I add a icon for spacemacs-buffer-mode 😂 : ryuslash/mode-icons#27

screen shot 2016-04-27 at 11 27 33 am

@ReneFroger
Copy link
Author

ReneFroger commented Apr 27, 2016

@codefalling but.... (setq mode-icons-change-mode-name nil) will also disable it in mode line, not only in Helm/Ido buffers. There will not be a icon displayed for the major mode. What will be the point of using mode icons after all then?

I was wondering if I could disable it for Helm/Ido buffers, not for also disabling for mode line?

@xcodebuild
Copy link
Contributor

@ReneFroger

(setq mode-icons-change-mode-name nil) will also disable it in mode line

Really? Everything works well in my spacemacs.

screen shot 2016-04-28 at 10 04 25 am

@ReneFroger
Copy link
Author

Could you provide the same working configuration with emacs -Q?

I don't use Spacemacs, so it seems another parts of code is conflicting with yours, which will 'cause' your code to working.

@ReneFroger
Copy link
Author

(setq mode-icons-change-mode-name nil) will also disable it in mode line

Any update on this?

@xcodebuild
Copy link
Contributor

@ReneFroger Sorry for late reply

With emacs -Q

(package-initialize)
(require 'mode-icons)
(require 'helm)
(setq mode-icons-change-mode-name nil)
(mode-icons-mode)

Everything works well
screen shot 2016-05-11 at 8 33 33 am

@CestDiego
Copy link
Contributor

bump. Is there some progress on whether we would like this? there is also now this https://github.com/domtronn/all-the-icons.el :)

@d12frosted
Copy link
Contributor

Wow, @CestDiego

That package looks super nice.

@atextor
Copy link
Contributor

atextor commented Oct 11, 2016

@ReneFroger I made mode-icons work in the mode-line but not the helm buffers by adding
(remove-hook 'helm-mode-hook 'mode-icons-mode)
(remove-hook 'helm-minibuffer-set-up-hook 'mode-icons-mode)
to my dotspacemacs/user-config.

@chriskaukis
Copy link
Contributor

This looks lovely! Has this been added to dev or master yet? I'm slightly on the new side to Emacs et al and can't seem to find this package in Spacemacs layers list.

@StreakyCobra
Copy link
Contributor

@chriskaukis In your .spacemacs file, add mode-icons to the dotspacemacs-additional-packages list, then this somewhere in dotspacemacs/user-config:

  ;; mode-icons
  (spacemacs|do-after-display-system-init
   (mode-icons-mode))

@mustaqimM
Copy link

mustaqimM commented May 6, 2017

@StreakyCobra Is there a way to do it with all-the-icons? I'm not getting it right

@bricewge
Copy link

bricewge commented May 6, 2017

The author of all-the-icons is working on a theme for spaceline with this icon set: domtronn/spaceline-all-the-icons.el.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid!

@github-actions github-actions bot added the stale marked as a stale issue/pr (usually by a bot) label Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Mailling list - Feature request New Package stale marked as a stale issue/pr (usually by a bot)
Projects
None yet
Development

No branches or pull requests