IJHack / QtPass

QtPass is a multi-platform GUI for pass, the standard unix password manager.
https://qtpass.org/
GNU General Public License v3.0
1.01k stars 158 forks source link

Invalid font color with Breeze Dark theme #340

Open vanyasem opened 6 years ago

vanyasem commented 6 years ago

I am running Arch GNU/Linux with KDE Plasma 5.11.4 (Qt 5.10.0)

I am using a dark theme (Breeze Dark), but unfortunately some fonts are rendered incorrectly and are impossible to read.

I tried both installing the package from repos and compiling it myself.

2017-12-30-024057_1602x968_scrot

daveol commented 6 years ago

The same issue arises by enabling the "Global Dark Theme" in gnome 3 on fedora screenshot from 2018-02-02 16-41-46

annejan commented 6 years ago

This should probably only use native theme colours instead of hardcoded colours.

Piraty commented 6 years ago

I can add to this. In most parts of qtpass my dark theme is respected in terms of color inversion, but not on "welcome screen". qtpass_dark-theme_fail Image shows qtpass-1.2.3 I suspect the text colors are hardcoded. yes, they are: https://github.com/IJHack/QtPass/blob/cc297302286a7e48d446531fe23224efb393cb5c/src/mainwindow.ui#L242-L251 and a few other localization/localization_xx_XX.ts files

annejan commented 6 years ago

Same issue with macOS Mojave screenshot 2018-06-28 at 10 24 09 Qt is working on some of the issues, the rest might be easily fixable with some style changes.

annejan commented 6 years ago

Fixed the HTML

screenshot 2018-06-28 at 11 26 37

I don't like the blue, but I like adding hardcoded styles even less . .

Testing on KDE Breeze Dark asap 😸

alxlg commented 5 years ago

Can you please use system colors and not hard code any color?

a-andreyev commented 4 years ago

Hello! Thanks for the handy app! Looks like it's possible just to drop the hardcoded color:#0000ff value at the src/configdialog.ui and src/mainwindow.ui to fix the issue, am I right? :thinking: Checking with Qt 5.13, KDE Plasma 5.16 (yet another arch fanboy), looks like html without specified color renders correctly according to the system-wide theme? (For example, at the config dialog version info)

Imgur Imgur

pheraph commented 4 years ago

I still see this problem on macOS (QtPass 1.3.2). 🧐

screen
Piraty commented 4 years ago

the dark blue hyperlink color seems to be inherited from system/qt, since i have that in quiet a lot applications (along: texstudio, quassel), so i suspect a general theming issue

annejan commented 4 years ago

This does not seem to be the case this time @Piraty Screenshot 2019-10-17 at 18 04 41 I have sanitised the HTML content of the start screen. Unfortunately QtCreator automagically places <span> the markup back . . @Piraty was right after all 😉

So I edited the mainwindow.ui file and replaced the HTML for clean HTML . .

Now it looks like this (macOS dark) . . Screenshot 2019-10-17 at 18 31 38

So no improvement there . . except for size and spacing 😞

annejan commented 4 years ago

I'm taking this up with Qt . . Along with the inherited transparency of context menu's . . Screenshot 2019-10-17 at 18 39 24