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

Needs theme with no background color #7

Open
Cretezy opened this issue May 2, 2021 · 5 comments
Open

Needs theme with no background color #7

Cretezy opened this issue May 2, 2021 · 5 comments

Comments

@Cretezy
Copy link

Cretezy commented May 2, 2021

I use a black background in my terminal, and the "dark" themes aren't really enough. Either having a black theme or one without background color (meaning using the terminal's background) would be ideal.

Great project!

@controversial
Copy link

Second this. I use a terminal with a translucent background:

Screen Shot 2021-05-01 at 8 26 09 PM

however, git-split-diffs adds an opaque background behind every character on the screen:

Screen Shot 2021-05-01 at 8 28 28 PM

It would be great if there were an option to render a theme without any background color on the non-highlighted lines (obviously, the lines highlighted as added or removed need to have a background color)

@banga
Copy link
Owner

banga commented May 2, 2021

Makes sense. The reason everything has a background is because I added support for alpha in colors, so that you could say e.g. line deletions are ff000022 which adds a reddish tinge to the background. This is especially useful if you have multiple layers like for highlighting inline changes.

Anyway, will think of a fix. Would you rather have a way to unset background colors in existing themes, or just have a separate theme without backgrounds? I'm guessing the former would be better so you can use any theme

@Cretezy
Copy link
Author

Cretezy commented May 2, 2021

Alternatively, a way to point to a custom .json theme file, so everyone can be happy 😃

@banga
Copy link
Owner

banga commented May 2, 2021

Oh yeah that might be better. You can hackishly do that right now by making theme-name a relative path, but there should be a better way

@controversial
Copy link

I hacked together a quick fix in #8!

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 a pull request may close this issue.

3 participants