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

Color scheme produces unexpected output since 1.33.0 #860

Closed
mzanibelli opened this issue Mar 20, 2024 · 3 comments
Closed

Color scheme produces unexpected output since 1.33.0 #860

mzanibelli opened this issue Mar 20, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@mzanibelli
Copy link

Hello!

I've been experiencing issues with my color scheme since 1.33.0. It happens inside Tmux. Both on Linux + Wayland + Foot (terminal emulator) and on MacOS + Kitty.

The selected_line disappears.

I tested a range of things including compiling the project myself and tinkering with it - to no avail. I'm seeing some similar issues here and there so there might be something wrong still.

Let me know if you need my complete configuration.

Broot 1.32.0:
image

Broot 1.36.1:
image

@mzanibelli mzanibelli added the bug Something isn't working label Mar 20, 2024
@mzanibelli
Copy link
Author

mzanibelli commented Jun 17, 2024

Found the cause. It's because I had NO_COLOR set to 1.
Note: there is a bug in Broot where setting NO_COLOR to 0 doesn't re-enable colorschemes. You need to unset the variable for it to work as expected.

@Canop
Copy link
Owner

Canop commented Jun 17, 2024

Regarding your note: I'm following this

Command-line software which adds ANSI color to its output by default should check for a NO_COLOR environment variable that, when present and not an empty string (regardless of its value), prevents the addition of ANSI color.

If you think NO_COLOR=0 should behave like the variable being absent, please fill an issue. It would probably be better IMO.

@mzanibelli
Copy link
Author

If you think NO_COLOR=0 should behave like the variable being absent, please fill an issue. It would probably be better IMO.

Actually I read the NO_COLOR spec again and your implementation is correct. That's why I edited my comment (maybe you read it via mail and didn't see the strikethrough text, no worries). I think it's good to follow the "official" behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants