Bug 157389 - klocalesample force black text
Summary: klocalesample force black text
Status: RESOLVED FIXED
Alias: None
Product: kcontrol
Classification: Unmaintained
Component: kcmlocale (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Hans Petter Bieker
URL:
Keywords:
: 176344 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-02-08 01:42 UTC by Carlo Segato
Modified: 2008-12-30 02:13 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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) :)