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

Invalid font color with Breeze Dark theme #340

Open
vanyasem opened this issue Dec 29, 2017 · 11 comments · Fixed by #476
Open

Invalid font color with Breeze Dark theme #340

vanyasem opened this issue Dec 29, 2017 · 11 comments · Fixed by #476

Comments

@vanyasem
Copy link

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
Copy link
Contributor

daveol commented Feb 2, 2018

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

@annejan
Copy link
Member

annejan commented Feb 2, 2018

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

@Piraty
Copy link

Piraty commented Jun 7, 2018

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:

QtPass/src/mainwindow.ui

Lines 242 to 251 in cc29730

<property name="html">
<string>&lt;!DOCTYPE HTML PUBLIC &quot;-//W3C//DTD HTML 4.0//EN&quot; &quot;http://www.w3.org/TR/REC-html40/strict.dtd&quot;&gt;
&lt;html&gt;&lt;head&gt;&lt;meta name=&quot;qrichtext&quot; content=&quot;1&quot; /&gt;&lt;style type=&quot;text/css&quot;&gt;
p, li { white-space: pre-wrap; }
&lt;/style&gt;&lt;/head&gt;&lt;body style=&quot; font-family:'Cantarell'; font-size:11pt; font-weight:400; font-style:normal;&quot;&gt;
&lt;p style=&quot; margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; font-weight:600; color:#333333;&quot;&gt;QtPass&lt;/span&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; color:#333333;&quot;&gt; is a GUI for &lt;/span&gt;&lt;a href=&quot;https://www.passwordstore.org/&quot;&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; text-decoration: underline; color:#4183c4; background-color:transparent;&quot;&gt;pass&lt;/span&gt;&lt;/a&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; color:#333333;&quot;&gt;, the standard unix password manager.&lt;/span&gt;&lt;/p&gt;
&lt;p style=&quot; margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; color:#333333;&quot;&gt;&lt;br /&gt;Please report any &lt;/span&gt;&lt;a href=&quot;https://github.com/IJHack/qtpass/issues&quot;&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; text-decoration: underline; color:#4183c4;&quot;&gt;issues&lt;/span&gt;&lt;/a&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; color:#333333;&quot;&gt; you might have with this software.&lt;/span&gt;&lt;/p&gt;
&lt;p style=&quot; margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;a href=&quot;https://qtpass.org/&quot;&gt;&lt;span style=&quot; font-family:'Helvetica Neue,Helvetica,Segoe UI,Arial,freesans,sans-serif'; font-size:13pt; text-decoration: underline; color:#4183c4; background-color:transparent;&quot;&gt;Documentation&lt;/span&gt;&lt;/a&gt;&lt;/p&gt;
&lt;p style=&quot; margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;&quot;&gt;&lt;a href=&quot;https://github.com/IJHack/qtpass&quot;&gt;&lt;span style=&quot; font-family:'Noto Sans'; font-size:12pt; text-decoration: underline; color:#4183c4;&quot;&gt;SourceCode&lt;/span&gt;&lt;/a&gt;&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
</property>
and a few other localization/localization_xx_XX.ts files

@annejan
Copy link
Member

annejan commented Jun 28, 2018

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
Copy link
Member

annejan commented Jun 28, 2018

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
Copy link

alxlg commented Jun 25, 2019

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

@a-andreyev
Copy link
Contributor

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? 🤔
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
Copy link

pheraph commented Oct 17, 2019

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

screen

@Piraty
Copy link

Piraty commented Oct 17, 2019

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
Copy link
Member

annejan commented Oct 17, 2019

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 annejan reopened this Oct 17, 2019
@annejan
Copy link
Member

annejan commented Oct 17, 2019

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

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

Successfully merging a pull request may close this issue.

7 participants