Skip to content

Comments

More icons with associated color and colorized long output #12

Closed
UnsaltedScholar wants to merge 2 commits intorockorager:mainfrom
UnsaltedScholar:add-color-to-long-output
Closed

More icons with associated color and colorized long output #12
UnsaltedScholar wants to merge 2 commits intorockorager:mainfrom
UnsaltedScholar:add-color-to-long-output

Conversation

@UnsaltedScholar
Copy link
Contributor

@UnsaltedScholar UnsaltedScholar commented Nov 5, 2025

This PR adds (optional) color to the long output and adds support for more file types with associated icon (with optional colors from language logos). Color choices based on eza colors along with personal config to match file name color to language icon/color.

Screenshot_20251104_215516 Screenshot_20251104_215635

@p00f
Copy link
Contributor

p00f commented Nov 5, 2025

oof one of us will have to rebase #11

@UnsaltedScholar
Copy link
Contributor Author

oof one of us will have to rebase #11

Hmmm... doesn't seem too bad I don't think. Will wait till repo owner provides comments on both PRs though in case they have different ideas.

@UnsaltedScholar UnsaltedScholar force-pushed the add-color-to-long-output branch from 77d31cd to fe36d55 Compare November 6, 2025 19:54
@rockorager
Copy link
Owner

Could you separate these out into different PRs?

  1. New icons (keep alphabetic sorted) + rebase (you are the chosen one to rebase :))
  2. Additional color to long output

I'm not sure I like all the color of the long output. I'm definitely not a fan of the icon color being the filename color. I'll have to think about the rest 🤔

@UnsaltedScholar
Copy link
Contributor Author

Will do, and I could add a command line option for the colored file name part if that would be more agreeable?

@UnsaltedScholar
Copy link
Contributor Author

@rockorager
Issues have been separated and I added a command line flag to make 2 optional. See #13 and #14

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.

3 participants