Bug 266434 - The color of layout name should be configurable
Summary: The color of layout name should be configurable
Status: RESOLVED FIXED
Alias: None
Product: kxkb
Classification: Unmaintained
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Unlisted Binaries Linux
: NOR minor
Target Milestone: ---
Assignee: Andriy Rysin
URL:
Keywords:
: 254346 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-16 11:22 UTC by Andrey Cherepanov
Modified: 2011-07-30 23:52 UTC (History)
1 user (show)

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

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Cherepanov 2011-02-16 11:22:21 UTC 
Version:           unspecified (using KDE 4.6.0) 
OS:                Linux

Layout name (without flag) display as black tixt with small glow. It look unreadable on dark Plasma theme and on flag.
There are two way (or their combination):
1. Use default fronend color from current Plasma theme;
2. Add color options for text and glow.

Reproducible: Always
Comment 1 Andriy Rysin 2011-02-19 20:17:00 UTC 
Git commit 73fc1bf4bafb80b7b4b718634d68dbc7797b2a8d by Andriy Rysin.
Committed on 19/02/2011 at 20:05.
Pushed by rysin into branch 'master'.

Use proper foreground color for text appropriate for current theme
BUG: 266434

M  +9    -11   kcontrol/keyboard/CMakeLists.txt     
M  +34   -4    kcontrol/keyboard/flags.cpp     
M  +16   -2    kcontrol/keyboard/flags.h     
M  +2    -0    kcontrol/keyboard/layout_tray_icon.cpp     
D  +0    -227  kcontrol/keyboard/utils.cpp     
D  +0    -35   kcontrol/keyboard/utils.h     

http://commits.kde.org/kde-workspace/73fc1bf4bafb80b7b4b718634d68dbc7797b2a8d
Comment 2 Andriy Rysin 2011-02-19 20:18:20 UTC 
I've pushed a fix into master, if you could try it there I'd appreciate it.
If it works I may merge it back into 4.6.
Comment 3 Andriy Rysin 2011-07-30 23:52:12 UTC 
*** Bug 254346 has been marked as a duplicate of this bug. ***