Bug 157389

Summary: klocalesample force black text
Product: [Unmaintained] kcontrol Reporter: Carlo Segato <brandon.ml>
Component: kcmlocaleAssignee: Hans Petter Bieker <bieker>
Status: RESOLVED FIXED    
Severity: normal CC: andresbajotierra
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Carlo Segato 2008-02-08 01:42:15 UTC
Version:            (using Devel)
Installed from:    Compiled sources
OS:                Linux

klocalesample force the text to be black and it doesn't look good with dark color scheme

  // Whatever the color scheme is, we want black text
  QPalette pal = palette();
  pal.setColor(QPalette::Disabled, QPalette::WindowText, Qt::black);
  pal.setColor(QPalette::Active, QPalette::WindowText, Qt::black);
  pal.setColor(QPalette::Inactive, QPalette::WindowText, Qt::black);
  setPalette(pal);
Comment 1 Dario Andres 2008-11-28 16:46:48 UTC
*** Bug 176344 has been marked as a duplicate of this bug. ***
Comment 2 Dario Andres 2008-11-28 16:48:34 UTC
From KDE WebSVN log:

-------------

Revision 64867 - (view) (download) (as text) (annotate) - [select for diffs]
Modified Sat Sep 23 21:45:18 2000 UTC (8 years, 2 months ago) by dfaure
Original Path: trunk/kdebase/kcontrol/locale/klocalesample.cpp
File length: 5266 byte(s)
Diff to previous 54871

Force colors of labels on the map to be always black, no matter which
color scheme is used (report by Lukas Tinkl).
Fixed crash on destruction (evil static QPixmap).

--------
This was introduced 8 years ago.
Is this actually necessary ?
Comment 3 Dario Andres 2008-12-30 02:13:31 UTC
Fixed by rev 895762 (http://websvn.kde.org/?view=rev&revision=895762) :)