Created attachment 125701 [details]
SMS messaging application with dark theme
When using a dark colour theme, the new KDE Connect SMS (1.4) application is very hard on the eye, I'm guessing because colours of the background of the messages is hard-coded, or at least, isn't related to the colour theme of the Plasma desktop.
Attaching a screenshot.
Step to reproduce:
- Set up a dark colour theme (like Breeze Dark or Arc Dark)
- Open the KDEConnect SMS application
- Enter a conversation, observe the text is hard to read (though certainly not unreadable!)
Operating System: openSUSE Tumbleweed 20200201
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.66.0
Qt Version: 5.14.1
Kernel Version: 5.4.14-1-default
OS Type: 64-bit
Created attachment 125806 [details]
Breeze Dark Theme Screenshot
Thanks for taking the time to report this!
This looks like an issue with your theme. It works well with Breeze Dark. None of the primary text or background colours are hard-coded.
In this case, it looks like those are messages you received, so they are using Kirigami.Theme.highlightColor and Kirigami.Theme.highlightedTextColor, though those only have a difficult relationship with the Qt theme colours. (These might not be the most-ideal colours to use for this purpuse but I couldn't find better ones)
The theme colours are defined in ChatMessage.qml: https://invent.kde.org/kde/kdeconnect-kde/blob/master/smsapp/qml/ChatMessage.qml
OK, I see. I forgot how it looked under Breeze Dark and should have re-tested it. It still feels a bit weird, but I guess designing for both light and dark theming is hard anyway. I'll see what I can tweak on my side.
FYI, it's also difficult to read using "Breeze Dark High Contrast". I believe a problem is that the text colour is black no matter the colour theme?
Sorry to come back to this, but could it be a bug in Kirigami?
Selected text (which corresponds to highlightedTextColor, at least in theory, right?) colour in Breeze is "#eff0f1" (so, white) and "#d3dae3" in my theme (Arc Dark). But in all screenshots published in this bug report, the text of received messages appears in black. I can't figure out where this black colour is coming from...
I don't know much about theming. My experience from other people has lead me to the same conclusion as you have come to: That there is something strange with the way Kirigami's theme and the Qt theme interact.
Could you add a minimal working example that reproduces the issue with kirigami?
Created attachment 138588 [details]
SMS Text invisible on Breath2Dark Color theme
I'm using Manjaro with the preinstalled Breath2 Dark theme and the text in the input field is white on white background -> I'm sure you could imagine that it's hard to read ;) Also the buttons are not visible, only on Mouseover a frame appears
@Christopher, sorry I missed your message before. The issue turned out to be a missing theme color option in Kirigami which was deprecated and removed. See the MR for details: https://invent.kde.org/network/kdeconnect-kde/-/merge_requests/393
@flyos: good news, the merge request above solves the problem for me and very likely will help you as well. I'm making this bug as a duplicate (not from particular merit of either one, just because I need to pick one)
*** This bug has been marked as a duplicate of bug 434791 ***
Ah sorry. I got confused. Please ignore my previous comment.
@Luxas, your problem is the one in the linked bug, and should be fixed.
@flyos, unfortunately I still don't have any idea how to help you. I am very unfamiliar with dealing with themes and how they interact with Kirigami/qml apps