Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: let users override icons#562

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
hasecilu wants to merge2 commits intomaster
base:master
Choose a base branch
Loading
fromfeat/override_icons
Draft

Conversation

@hasecilu
Copy link
Collaborator

@haseciluhasecilu commentedFeb 27, 2025
edited
Loading

Previously if you wanted to change one icon across several
extensions/filenames you needed to manually override each
instance, now you can target specific icons to simplify
config file.

Basically I'm implementing this feature to match icons to
eza and lazygit, and simplify setup to avoid big config file.

config file used

BeforeAfter
imageimage

config file used

return {"nvim-tree/nvim-web-devicons",event="VimEnter",opts= {override_by_icon= {      [""]= {icon=""},      [""]= {icon=""},    },  },}

Probably it's also a good idea to:

  • override color

Previously if you wanted to change one icon across severalextensions/filenames you needed to manually override eachinstance, now you can target specific icons to simplifyconfig file.
@alex-courtis
Copy link
Member

That's rather nice... we should have done this years ago.

My initial thought was dark/light variants, but that's a bigger task#540

@haseciluhasecilu requested a review fromgegouneMarch 2, 2025 21:09
@alex-courtis
Copy link
Member

Is this good to go@hasecilu ? Can we merge?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@alex-courtisalex-courtisalex-courtis left review comments

@gegounegegouneAwaiting requested review from gegoune

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hasecilu@alex-courtis

[8]ページ先頭

©2009-2025 Movatter.jp