Bug 310389 - Plasma::CheckBox() using colors from color-schemes instead of plasma theme
Summary: Plasma::CheckBox() using colors from color-schemes instead of plasma theme
Status: RESOLVED DUPLICATE of bug 263979
Alias: None
Product: plasma4
Classification: Unmaintained
Component: visuals (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-19 23:31 UTC by Hrvoje Senjan
Modified: 2013-01-29 00:31 UTC (History)
5 users (show)

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

Attachments
Screenshot (157.10 KB, image/png)
2012-11-20 12:44 UTC, Hrvoje Senjan 		Details
Box (162.69 KB, image/png)
2012-11-20 13:31 UTC, Hrvoje Senjan 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Hrvoje Senjan 2012-11-19 23:31:25 UTC 
But it is using indicated color from colorscheme, just not with plasma-themes colors.

Reproducible: Always

Steps to Reproduce:
1. Change BackgroundNormal from plasma-theme colors file

Actual Results:  
Button color doesn't change

Expected Results:  
Should change, as it behaves that way with regular colorscheme. Also ForegroundNormal is correctly changed.
Comment 1 Lamarque V. Souza 2012-11-20 11:24:04 UTC 
Reassigning because Plasma NM does not read the plasma theme configuration, that is automatically handled by kdelibs.
Comment 2 Sebastian Kügler 2012-11-20 12:29:30 UTC 
Does it change after a Plasma restart? How do you change it, do you edit the file by hand?
Comment 3 Hrvoje Senjan 2012-11-20 12:36:30 UTC 
Sebastian, thanks for the response, yes, i always restart plasma after i do some changes. 
Yes, i am changing it by hand, after i check the color in colors KCM.
Comment 4 Hrvoje Senjan 2012-11-20 12:44:15 UTC 
Created attachment 75377 [details]
Screenshot
Comment 5 Sebastian Kügler 2012-11-20 13:13:18 UTC 
Hm, off-hand, I don't see what should be different (didn't spend too much time guessing, either ;)). Could you point me at it?
Comment 6 Hrvoje Senjan 2012-11-20 13:22:50 UTC 
:-)
The box is using the "Button background" color from colorscheme (currently, for me 236,235,235) and not what is in the colors file (which i currently set to 0,192,0)
Comment 7 Sebastian Kügler 2012-11-20 13:30:05 UTC 
Which box exactly?
Comment 8 Hrvoje Senjan 2012-11-20 13:31:27 UTC 
Created attachment 75379 [details]
Box
Comment 9 Sebastian Kügler 2012-11-20 13:40:39 UTC 
I still think that I'm flying blind, or being dense here.

Could you tell me exactly what it should look like, and what is wrong?

(As in, "the checkmark is grey, but should be pink, the background right behind the checkmark is grey, but should be purple. The shadow is fine."
Comment 10 Hrvoje Senjan 2012-11-20 18:20:55 UTC 
(In reply to comment #9)
... the background right
> behind the checkmark is xxx, but should be yyy. The shadow is fine."

That's it :-)
Comment 11 Lamarque V. Souza 2013-01-17 12:56:13 UTC 
Why change the product of this bug back to "Network Management"? I cannot do anything to fix it from Plasma NM, it should be fixed in plasma libs. Besides, removing plasma-bugs@kde.org  from the assignee/CC fields you are basically removing the ones that could fix this bug from reading comments about it.
Comment 12 David Edmundson 2013-01-17 14:03:51 UTC 
Apologies, I did not read the bug history correctly and I wanted to move it out of plasma component "general".

Can I suggest changing the bug title to specify where the bug actually lies.
Comment 13 Hrvoje Senjan 2013-01-20 15:46:49 UTC 
Issue here is the same as in bug #263979
Comment 14 Hrvoje Senjan 2013-01-22 10:52:53 UTC 
Made a review request https://git.reviewboard.kde.org/r/108525/
Comment 15 Sebastian Kügler 2013-01-22 13:17:53 UTC 
Fixed by 5566777af99bbb098d221bfdb384734f3b44ad50 in kdelibs.
Comment 16 Christoph Feck 2013-01-29 00:31:49 UTC 

*** This bug has been marked as a duplicate of bug 263979 ***