https://bugs.kde.org/show_bug.cgi?id=528107

            Bug ID: 528107
           Summary: Colors KCM: "Normal text" row in light scheme previews
                    is drawn in the running dark theme's text color (white
                    on white)
    Classification: Applications
           Product: systemsettings
      Version First 6.8.2
       Reported In:
          Platform: CachyOS
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: kcm_colors
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected],
                    [email protected]
  Target Milestone: ---

Created attachment 197889
  --> https://bugs.kde.org/attachment.cgi?id=197889&action=edit
Screenshot: in the Breeze Hell and Breeze Klassisch previews the "Normaler
Text" row is white on white

SUMMARY
In System Settings > Colors, the "Normal text" row in the Breeze Light and
Breeze Classic previews is white on a white background when a dark global theme
is active.

STEPS TO REPRODUCE
1. Plasma 6.8.2 with a dark global theme, widget style Breeze (Union) or
Breeze.
2. Open System Settings > Colors.

OBSERVED RESULT
In the Breeze Light and Breeze Classic previews the "Normal text" row is white
on a white background and only readable on hover. The "Highlighted text" and
"Disabled text" rows look fine.

EXPECTED RESULT
The row uses the previewed scheme's text color.

SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.8.2 (plasma-desktop/systemsettings 6.8.2-1.1)
Distribution: CachyOS

ADDITIONAL INFORMATION
- Reproduces with both Breeze (Union) and classic Breeze.
- Debugging for Breeze Light: the scheme's palette.text and
Kirigami.Theme.textColor (on the preview Frame and on the ItemDelegate itself)
are all #232629, which is correct. But the Label inside the ItemDelegate's
contentItem ends up with #fcfcfc, the running dark theme's text color. So the
desktop style's inner Label does not pick up the Kirigami.Theme override set on
the preview Frame.
- Did not help: palette.text / palette.windowText on the delegate;
Kirigami.Theme.inherit / textColor on the delegate.
- Works: giving the first ItemDelegate in kcms/colors/ui/main.qml its own
contentItem Label with an explicit color (kcm.tinted(model.palette.text, ...))
and wrapMode: Text.Wrap. Tested with all three bundled schemes under both
styles.
- Possible deeper cause: Kirigami.Theme inheritance through
ItemDelegate.contentItem in qqc2-desktop-style.
- Possibly related to the preview problems in Bug 528106 (text overflowing the
thumbnails).

--- a/kcms/colors/ui/main.qml
+++ b/kcms/colors/ui/main.qml
@@ -269,7 +269,16 @@ KCM.GridViewKCM {

                         QtControls.ItemDelegate {
                             width: parent.width
+                            id: normalTextDelegate
                             text: i18n("Normal text") +
listPreviewColumn.demoText(model.palette)
+                            contentItem: QtControls.Label {
+                                text: normalTextDelegate.text
+                                font: normalTextDelegate.font
+                                color: kcm.tinted(model.palette.text,
kcm.accentColor, model.tints, model.tintFactor)
+                                verticalAlignment: Text.AlignVCenter
+                                wrapMode: Text.Wrap
+                            }
                             activeFocusOnTab: false
                         }

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to