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

Disabling Background Color #146

Closed
h3x4g0ns opened this issue Apr 22, 2022 · 6 comments · Fixed by #168
Closed

Disabling Background Color #146

h3x4g0ns opened this issue Apr 22, 2022 · 6 comments · Fixed by #168
Labels

Comments

@h3x4g0ns
Copy link

Is it possible to disable to background color when cmatrix runs? I have a color theme applied in the terminal so when cmatrix runs, there's a gray background underneath.

Screenshot_2022-04-21-05-59-22_3440x1440

@abishekvashok
Copy link
Owner

Can you try building with make and using code from the development branch please?

@nikoof
Copy link
Contributor

nikoof commented Oct 25, 2022

It seems the background color depends on whether use_default_colors is used. And that depends on whether HAVE_USE_DEFAULT_COLORS is defined, which the autoconf script does, but not the CMakeLists.txt file, so I figure that the package you're using was configured with cmake and that's why it's missing this feature.

Checking for the function and defining that macro in the cmake script should solve it.

@lilly-lizard
Copy link

just tried building this with default autoconf config and there's no background which is great! just a fyi in case this could be closed.

@abishekvashok
Copy link
Owner

Yes, I think it works via autotools and doesn't work with cmake. I think we could use the commit authored by @nikoof on his fork. @nikoof would you like to make a pr from your fork?

@nikoof
Copy link
Contributor

nikoof commented Feb 28, 2023

Will do!

@abishekvashok
Copy link
Owner

Seems like #168 closes this, let me know if it doesn't :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants