-
Notifications
You must be signed in to change notification settings - Fork 3
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 an (optional) flag to disable icons #3
Comments
Hi @rashil2000 |
And here I thought this was no longer maintained (as is the case with most PowerShell modules sadly). Sweet update! Thanks for the addition. I have two issues to report though. One is the color of icons. Since PowerColorLS uses Terminal-Icons as a dependency, it should out the same colors as given by Terminal-Icons, right? Or is the colorscheme different? Some of the colors don't match and this causes trouble on Light themed backgrounds :( See the pictures for reference. I think the difference is only in the default color (not for specific hard-coded folder colors). The other one is about directory sizes. Since it's not related to icons/colors, I'll open a separate issue for that #5 |
Hi again! |
@gardebring The screenshots provided above compare the default Terminal-Icons theme with PowerColorLS colors. So, in my opinion, the module dependency is getting loaded correctly. However, the difference arises while rendering the default color (i.e. files that do not have any specific color associated with them in the devblackops theme). For Terminal-Icons, the default fallback color is white (which gets converted to black in my Windows Terminal theme when I use Light mode), but the default fallback color for PowerColorLS seems to be yellow (which somehow doesn't convert to a darker shade when using a light mode). Ideally, PowerColorLS should default to white, just like Terminal-Icons does, but something seems to be off, and I can't figure out what. |
@rashil2000 : |
My Windows Terminal config file looks like this: "schemes": [
{
"name": "Bluloco Light",
"black": "#777777",
"red": "#c90e42",
"green": "#21883a",
"yellow": "#d54d17",
"blue": "#1e44dd",
"purple": "#6d1bed",
"cyan": "#1f4d7a",
"white": "#000000",
"brightBlack": "#999999",
"brightRed": "#fc4a6d",
"brightGreen": "#34b354",
"brightYellow": "#b89427",
"brightBlue": "#1085d9",
"brightPurple": "#c00db3",
"brightCyan": "#5b80ad",
"brightWhite": "#1d1d22",
"background": "#ffffff",
"foreground": "#2a2c33"
},
{
"name": "Bluloco Dark",
"black": "#4a505d",
"red": "#f81141",
"green": "#23974a",
"yellow": "#fd7e57",
"blue": "#285bff",
"purple": "#8c62fd",
"cyan": "#366f9a",
"white": "#a1b8df",
"brightBlack": "#61697a",
"brightRed": "#fc4a6d",
"brightGreen": "#37bd58",
"brightYellow": "#f6be48",
"brightBlue": "#199ffd",
"brightPurple": "#fc58f6",
"brightCyan": "#50acae",
"brightWhite": "#ffffff",
"background": "#1e2127",
"foreground": "#abb2bf"
}
] How/where do I add a conversion? PowerColorLS seems to be overriding these... Also, for files/directories that don't have a color defined, Terminal-Icons just prints them as normal text. I think this is why Windows Terminal is able to convert the text color. |
For the time being, I just changed both EEEEEE and EEEE8B to 888888 (grey), which is visible on both black and white :) |
Great. Should I close this? |
Yes sure! I'll do it. |
Hi!
It's so nice that PowerColorLS provides so many flags for output. Is it possible to have a flag to display the items with colors but without icons? Some terminals don't have icon support and I currently use https://github.com/JRJurman/PowerLS to have unix-like ls there.
It'd be great if I just install one module instead of two.
Thanks!
The text was updated successfully, but these errors were encountered: