Bug 310389

Summary:	Plasma::CheckBox() using colors from color-schemes instead of plasma theme
Product:	[Plasma] plasma4	Reporter:	Hrvoje Senjan <hrvoje.senjan>
Component:	visuals	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	kde, lamarque, plasma-bugs, sebas, wstephenson
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Screenshot Box

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 ***