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

feat: LS_ICONS for icon overrides #909

Closed
pagerc opened this issue Mar 26, 2024 · 4 comments
Closed

feat: LS_ICONS for icon overrides #909

pagerc opened this issue Mar 26, 2024 · 4 comments

Comments

@pagerc
Copy link

pagerc commented Mar 26, 2024

This is a continuation of the unaddressed exa issue: ogham/exa#559

LS_ICONS would make a good corollary to LS_COLORS for overriding eza's internal file extension to icon mappings.
Example derived from (ogham/exa#559 (comment)):

LS_ICONS="*.rb=e21e:*.vim=e62b:*.sh=f292:"
@cafkafk
Copy link
Member

cafkafk commented Mar 27, 2024

We'd love to see a PR for this if anyone has time :D

gierens added a commit that referenced this issue Mar 30, 2024
@gierens
Copy link
Member

gierens commented Mar 30, 2024

Since I was just in that part of the code the other day I thought I give it a shot, see the linked PR draft :D

gierens added a commit to PThorpe92/eza that referenced this issue Mar 31, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
@gierens
Copy link
Member

gierens commented Mar 31, 2024

I ported my draft from #909 to #840 ... so overriding icons will be a feature of the config file once that is merged.

@fgclue
Copy link

fgclue commented Apr 2, 2024

if this gets added i'm going to finally switch that folder icon with the better one

hasecilu pushed a commit to hasecilu/eza that referenced this issue Apr 7, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
hasecilu pushed a commit to hasecilu/eza that referenced this issue Apr 7, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
PThorpe92 pushed a commit to PThorpe92/eza that referenced this issue Aug 30, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
erwinvaneijk pushed a commit to erwinvaneijk/eza that referenced this issue Oct 1, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
tertsdiepraam pushed a commit to tertsdiepraam/eza that referenced this issue Oct 10, 2024
This basically ports the PR eza-community#914 and thus also resolves eza-community#909

Signed-off-by: Sandro-Alessio Gierens <[email protected]>
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

Successfully merging a pull request may close this issue.

4 participants