From decd95c446e2df9e269781c7ea0ee738c2f9007b Mon Sep 17 00:00:00 2001 From: un1versal Date: Tue, 27 Oct 2015 13:07:49 +0000 Subject: OptionsWidget_servers: connect button window unfocus state color adjustments + add comments @blindsight stumbled over a design omission since cd37db31. When the KVIrc window was unfocused the Active button/text color schemes were maybe left up to Qt. The comments should hopefully benefit future changes. This PR adds slight fine-tuning on the button unfocused state color scheme should change both button/text color not just in one or the other in a manner that reflects something happened but doesnt fundamentally change the status of the button.--- src/modules/options/OptionsWidget_servers.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/modules/options/OptionsWidget_servers.cpp b/src/modules/options/OptionsWidget_servers.cpp index 8f04e69f4..3145d46df 100644 --- a/src/modules/options/OptionsWidget_servers.cpp +++ b/src/modules/options/OptionsWidget_servers.cpp @@ -1404,12 +1404,18 @@ OptionsWidget_servers::OptionsWidget_servers(QWidget * parent) QPalette pal(QColor(0,0,0)); + /* QPalette::active defines the colors of the button/text when the KVIrc window has focus + Keep button/text color in a specturm that indicates a connect action is possible */ pal.setColor(QPalette::Active,QPalette::Button,QColor(0,135,0)); pal.setColor(QPalette::Active,QPalette::ButtonText,QColor(245,245,245)); - pal.setColor(QPalette::Inactive,QPalette::Button,QColor(0,135,0)); - pal.setColor(QPalette::Inactive,QPalette::ButtonText,QColor(0,0,0)); + /* QPalette::Inactive defines the colors of the button/text when the KVIrc window looses focus + Keep button/text color the same as QPalette::Active but use different shades */ + pal.setColor(QPalette::Inactive,QPalette::Button,QColor(50,135,50)); + pal.setColor(QPalette::Inactive,QPalette::ButtonText,QColor(225,225,225)); + /* QPalette::Disabled defines the colors of the button/text when for instance network is selected + Keep button/text color in a specturm that indicates no possible action is possible */ pal.setColor(QPalette::Disabled,QPalette::Button,QColor(100,100,100)); pal.setColor(QPalette::Disabled,QPalette::ButtonText,QColor(180,180,180)); -- cgit v1.3.1-10-gc9f91