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

Add Perl module and test script icons #880

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mjgardner
Copy link

No description provided.

@ariasuni
Copy link
Collaborator

Wikipedia lists as Perl extensions: plx, pl, pm, xs, t, pod. Right now, only pl is taken into account, and you’re proposing to add pm and t. What about the other extensions, are they legacy or too specific?

plx seems legacy, pod is documentation so another icon is probably more appropriate, and xs seems like something that could use the same icon as other Perl files.

@ariasuni
Copy link
Collaborator

ariasuni commented Jun 28, 2021

Because exa doesn’t try to limit itself to what ls does, and that a lot of people find icons nice and/or useful, and it’s optional so nobody has to use it if they don’t like them. File managers have icons, I don’t see why a file lister can’t have them too.

@ariasuni
Copy link
Collaborator

It’s characters that display correctly if you install a font from Nerd Fonts. I need to document that properly, as I wrote in another issue.

@ariasuni
Copy link
Collaborator

Hi @mjgardner, did you see my comment?

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 this pull request may close these issues.

2 participants