Bug 495291

Summary: SwipeListItem has the wrong background color
Product: [Frameworks and Libraries] frameworks-kirigami Reporter: Debanga Sarma <sarma.deb>
Component: generalAssignee: kdelibs bugs <kdelibs-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, nate, notmart, soumyadeepghosh2004
Priority: NOR    
Version: 6.7.0   
Target Milestone: Not decided   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In: 6.9
Sentry Crash Report:
Attachments: Screenshot of the issue.

Description Debanga Sarma 2024-10-24 10:18:17 UTC
Created attachment 175177 [details]
Screenshot of the issue.

SUMMARY

The settings page in the Discover application is unreadable when choosing a Dark color scheme.

STEPS TO REPRODUCE
1. Select a Dark color scheme (I chose a color scheme named Carl which I downloaded but you can observe the same in the default Breeze Dark color scheme).
2. Open Discover.
3. Navigate to the settings page.

OBSERVED RESULT
All the entries have a white background which makes the text inside them unreadable.

EXPECTED RESULT
The text should be readable.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora Linux 40
KDE Plasma Version: 6.2.1
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.2

ADDITIONAL INFORMATION
Here is the output from `kinfo`
```bash
Operating System: Fedora Linux 40
KDE Plasma Version: 6.2.1
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.2
Kernel Version: 6.11.3-200.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 8 × Intel® Core™ i5-8350U CPU @ 1.70GHz
Memory: 23.3 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics 620
```
Comment 1 Soumyadeep Ghosh 2024-11-03 06:12:22 UTC
The text shows properly only in Breeze Dark color scheme. I use Breeze Black colorscheme and it does the same with this color. So, I think it's due to the contrast. Not sure, if it can actually be fixed.
Comment 2 Nate Graham 2024-11-13 23:42:30 UTC
This is a bug in Kirigami.SwipeListItem; it shows the wrong background color, which interacts poorly with some color schemes. It's reproducible in ./tests/swipeListItemTest.qml

It's not visible with Breeze Light, but quite visible with Breeze Dark.
Comment 3 Bug Janitor Service 2024-11-13 23:54:34 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1662
Comment 4 Bug Janitor Service 2024-11-14 10:50:49 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/430
Comment 5 Marco Martin 2024-11-14 13:49:58 UTC
Git commit d062e78a556e32578bfd74b9aa12120f7033e539 by Marco Martin.
Committed on 14/11/2024 at 10:50.
Pushed by mart into branch 'master'.

Add a SwipeDelegate

Adds the missing SwipeDelegate component, so that it can have
the standard ItemDelegate background, units and so on.
This is used by Kirigami SwipeListItem which used to have
wrong backgrounds especially visible in dark mode

Also, all components should be implemented here, no matter what
Related: bug 6, bug 9

A  +99   -0    org.kde.desktop/SwipeDelegate.qml     [License: GPL(v2.0+) LGPL(v3.0)]

https://invent.kde.org/frameworks/qqc2-desktop-style/-/commit/d062e78a556e32578bfd74b9aa12120f7033e539