ryuslash / mode-icons

Show icons instead of mode names
http://projects.ryuslash.org/mode-icons/
159 stars 28 forks source link

c-mode problems #18

Closed Dickby closed 8 years ago

Dickby commented 8 years ago

I installed font-mfizz-2.3.0 and the C symbol is #xf108 not #xf107.

I don't know why but with my emacs-config, modes like C, C++ and Java have the lighter-extension "/lh" (unlike "/l" in your code) so icons for these modes are not working.

mattfidler commented 8 years ago

For me, the C mode is displaying correctly with #xf107

image

mattfidler commented 8 years ago

If I change to #xf108, I get the following display

image

mattfidler commented 8 years ago

The lighter C++/lh should be replaced. Let me know if it works for you.

Thank you for your bug report.

What is displaying for the C icon for you?

Dickby commented 8 years ago

it works now for modes with "/lh" extension, but all of the mfizz symbols are wrong!?

The Symbol for C is a Bomb.

ryuslash commented 8 years ago

Looking at the mfizz website the code is definitely #xf107, #xf108 is the cassandra icon as @mattfidler shows. It is strange it's different for you. Where did you install mfizz from?

mattfidler commented 8 years ago

Also goto the buffer *scratch* and then type M-: (insert-char #xf107)

They type M-x describe-char

You should get some font information that may help diagnose the issue: image

Dickby commented 8 years ago

It's probably the way I installed mfizz (I made an arch-package with makefontpkg from the .ttf-file).

mattfidler commented 8 years ago

Hm. I don't think that should change what characters are displayed, but I'm too unfamiliar with the makefontpkg. Perhaps you can copy the font to ~/.fonts. Many distributions set this directory up to install fonts from.

mattfidler commented 8 years ago

In the most recent unreleased commit, you no longer need the fonts installed. However, the icons look better if you can get the fonts installed. So, I would remove the font to see if it displays the C icon for you.

mattfidler commented 8 years ago

I assume this works now. If not let me know.