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

Tilda searchbar does not respect dark themes #437

Closed
kanehekili opened this issue Oct 1, 2020 · 5 comments
Closed

Tilda searchbar does not respect dark themes #437

kanehekili opened this issue Oct 1, 2020 · 5 comments

Comments

@kanehekili
Copy link

The background color of the searchbar is white - always. As long as you're using a light/gray theme is doesn't hurt, but it looks weird with a dark theme.

GTK introspection shows that the background color has been defined locally for the box with ID=search.
image

@lanoxx
Copy link
Owner

lanoxx commented Oct 25, 2020

You are right, that has been defined here: https://github.com/lanoxx/tilda/blob/master/src/tilda.c#L533:

style = "#search{background:#fff;}";

I agree that this is not a suitable solution to handle dark themes.

@lanoxx lanoxx closed this as completed in 157e2b7 Oct 25, 2020
@lanoxx
Copy link
Owner

lanoxx commented Oct 25, 2020

Here is some background on this issue. The problem has actually been bugging me for many years, and the addition of the white background to the search box was just a poor work around for the issue. This workaround worked well for light themes but obviously fails for dark themes.

I spend some time on this today and finally found a solution which I believe works a bit better than the hard-coded CSS rule that we used before. I am closing this issue because I believe the change in commit 157e2b7 solves the problem. If not, then please reopen this issue with more details.

@kanehekili
Copy link
Author

I've submitted bug reports on Arch and Ubuntu, to get it upstream. Thanks for your help

@lanoxx
Copy link
Owner

lanoxx commented Feb 28, 2021

Not sure what you mean by "get it upstream". This project is the upstream project. The code is already in the master branch, but I have made a new release yet, so this will take some time to appear in Ubuntu.

@kanehekili
Copy link
Author

My mistake. I've meant "downstream". Arch has already reacted, debian (surprise) not. Since I'm using Tilda on a daily base I'd like to thank you for your excellent work.

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

No branches or pull requests

2 participants