-
Notifications
You must be signed in to change notification settings - Fork 794
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
Highlighted search results are unreadable when using a dark theme #1203
Comments
Definitely a bug, but may be tricky to fix. The highlighting is inspired by Windows Explorer, but it's a simple overlay. To render properly the color of the underlying letters needs to be reversed. Will check how Windows does it with the new dark theme. |
Maybe a simpler fix would be to change the opacity of the yellow overlay to something like |
@kaygdev I check the old PR, the relevant change from it should work, but need to detect the dark theme. Need to check if Qt has anything for that. |
In this PR I have introduce a UI Style option in Preferences/General. At this option, you can choose between the system default, the dark style and a custom CSS. The dark style apply a shipped CSS file and I see the drawback of it. In another project I made good experiences with the QPalette in combination with the fusion style for a dark UI. I think this can also be a good solution for zeal. |
It's something I'd definitely be interested to look at. Relying on unsupported 3rd party style is risky, but if we can provide a relatively good experience with the Fusion style, that should be better than nothing. |
Same problem here til now for Zeal 0.6.2-dev.212+8a018dc, is any fixed patchs can merged to master? |
Same here :) learning to live with it though |
Any updates on this, or any workarounds? I see you can point to a custom CSS in the settings, but not sure if that applies to the sidebar too, or what classes to override. |
what @mttkay said :-) What would be the simplest css fix for this? |
@trollixx Thank you for this wonderful application. |
This issue is highly theme dependent. Proper fix requires quite a bit more complex logic in how search item delegate renders text. A temporary workaround for this would be switching to another UI style. For me Fusion style renders everything quite nicely. To test it, run Zeal with |
Any ideas how to pass command line args when zeal is installed as a flatpak? I've used flatseal to set env vars such as for Qt, but it does not seem to control command line arguments. I also couldn't locate a desktop file for the zeal flatpak. 🤔 |
@mttkay I think it is |
I'm running Zeal under Gnome using the
adwaita-dark
theme, and have this environment variable set to make Qt applications match the theme:export QT_STYLE_OVERRIDE=Adwaita-Dark
When I search for something, the highlighted part of the list of results is unreadable, as seen in the screenshot below:
The text was updated successfully, but these errors were encountered: