From 6e2254c13d631f28feb5c34cba755b3019331f67 Mon Sep 17 00:00:00 2001 From: Janek Bevendorff Date: Sat, 25 Feb 2017 16:09:26 +0100 Subject: [PATCH 1/3] Don't fall back to ugly Qt close icon --- src/gui/KMessageWidget.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gui/KMessageWidget.cpp b/src/gui/KMessageWidget.cpp index e2c22574c..522a72f54 100644 --- a/src/gui/KMessageWidget.cpp +++ b/src/gui/KMessageWidget.cpp @@ -20,6 +20,8 @@ */ #include "KMessageWidget.h" +#include "core/FilePath.h" + #include #include #include @@ -89,7 +91,7 @@ void KMessageWidgetPrivate::init(KMessageWidget *q_ptr) QAction *closeAction = new QAction(q); closeAction->setText(KMessageWidget::tr("&Close")); closeAction->setToolTip(KMessageWidget::tr("Close message")); - closeAction->setIcon(q->style()->standardIcon(QStyle::SP_DialogCloseButton)); + closeAction->setIcon(FilePath::instance()->icon("actions", "dialog-close")); QObject::connect(closeAction, SIGNAL(triggered(bool)), q, SLOT(animatedHide())); From 84f2520924ce1f91f60d1c82bf865c9ed0ff5172 Mon Sep 17 00:00:00 2001 From: Janek Bevendorff Date: Sat, 25 Feb 2017 16:18:36 +0100 Subject: [PATCH 2/3] Fix message widget style on OS X --- src/gui/KMessageWidget.cpp | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/gui/KMessageWidget.cpp b/src/gui/KMessageWidget.cpp index 522a72f54..a6bfa04bc 100644 --- a/src/gui/KMessageWidget.cpp +++ b/src/gui/KMessageWidget.cpp @@ -98,6 +98,12 @@ void KMessageWidgetPrivate::init(KMessageWidget *q_ptr) closeButton = new QToolButton(content); closeButton->setAutoRaise(true); closeButton->setDefaultAction(closeAction); +#ifdef Q_OS_MAC + closeButton->setStyleSheet("QToolButton { background: transparent;" + "border-radius: 2px; padding: 3px; }" + "QToolButton::hover, QToolButton::focus {" + "border: 1px solid rgb(90, 200, 250); }"); +#endif q->setMessageType(KMessageWidget::Information); } @@ -287,7 +293,11 @@ void KMessageWidget::setMessageType(KMessageWidget::MessageType type) } // Colors +#ifdef Q_OS_MAC + fg = palette().light().color(); +#else fg = palette().highlightedText().color(); +#endif bg0 = bg1.lighter(110); bg2 = bg1.darker(110); border = darkShade(bg1); From 9ba88838ba99ab104dc0149006089045327e2737 Mon Sep 17 00:00:00 2001 From: Janek Bevendorff Date: Sat, 25 Feb 2017 21:42:55 +0100 Subject: [PATCH 3/3] Use white close icon and add more padding to message widget --- .../16x16/actions/message-close.png | Bin 0 -> 457 bytes .../22x22/actions/message-close.png | Bin 0 -> 457 bytes share/icons/svg/message-close.svg | 65 ++++++++++++++++++ src/gui/KMessageWidget.cpp | 7 +- 4 files changed, 67 insertions(+), 5 deletions(-) create mode 100644 share/icons/application/16x16/actions/message-close.png create mode 100644 share/icons/application/22x22/actions/message-close.png create mode 100644 share/icons/svg/message-close.svg diff --git a/share/icons/application/16x16/actions/message-close.png b/share/icons/application/16x16/actions/message-close.png new file mode 100644 index 0000000000000000000000000000000000000000..4b2f9ca4d7840c5c8a181bddcf779c88f1bee2b7 GIT binary patch literal 457 zcmV;)0XF`LP)T(Rc?!;7lrb z3nc1N-70%V!L6SQddQ@yo*9Ptl3C2WuCITptHvoJU|-UOq%%o}k~-d+N_v&_AZh5F zo5dI*unr7>FN&1_eV|?hZ2^xZfT3Q1gBY{}Oe;WBeFAN3`%hayQ%!(c7CeXyTI4_$ zeoUy|D*LP_Y63KX;|P4=y@ml#fTn;OBk%_}^%EZf-+_0M@#aSSm4KmPtXAa#>B_Bd zmhJ?+M@$P|1`6<|$X2pi9ZSLXvzqUeDo2Te_fWt8dKC91p_?S{R=~Ak29`H)FOj6)tGD(^6)mx)>l;@K+C?Y%SJ!00DUIb=!0YF zv7S&%GoTCXh8&kOK~vJXf2^PBNYcnTmss(Cf&p^D#659M00000NkvXXu0mjfp<%a# literal 0 HcmV?d00001 diff --git a/share/icons/application/22x22/actions/message-close.png b/share/icons/application/22x22/actions/message-close.png new file mode 100644 index 0000000000000000000000000000000000000000..4b2f9ca4d7840c5c8a181bddcf779c88f1bee2b7 GIT binary patch literal 457 zcmV;)0XF`LP)T(Rc?!;7lrb z3nc1N-70%V!L6SQddQ@yo*9Ptl3C2WuCITptHvoJU|-UOq%%o}k~-d+N_v&_AZh5F zo5dI*unr7>FN&1_eV|?hZ2^xZfT3Q1gBY{}Oe;WBeFAN3`%hayQ%!(c7CeXyTI4_$ zeoUy|D*LP_Y63KX;|P4=y@ml#fTn;OBk%_}^%EZf-+_0M@#aSSm4KmPtXAa#>B_Bd zmhJ?+M@$P|1`6<|$X2pi9ZSLXvzqUeDo2Te_fWt8dKC91p_?S{R=~Ak29`H)FOj6)tGD(^6)mx)>l;@K+C?Y%SJ!00DUIb=!0YF zv7S&%GoTCXh8&kOK~vJXf2^PBNYcnTmss(Cf&p^D#659M00000NkvXXu0mjfp<%a# literal 0 HcmV?d00001 diff --git a/share/icons/svg/message-close.svg b/share/icons/svg/message-close.svg new file mode 100644 index 000000000..44b643072 --- /dev/null +++ b/share/icons/svg/message-close.svg @@ -0,0 +1,65 @@ + + + + + + image/svg+xml + + + + + + + + + + + diff --git a/src/gui/KMessageWidget.cpp b/src/gui/KMessageWidget.cpp index a6bfa04bc..b88c3bc12 100644 --- a/src/gui/KMessageWidget.cpp +++ b/src/gui/KMessageWidget.cpp @@ -91,7 +91,7 @@ void KMessageWidgetPrivate::init(KMessageWidget *q_ptr) QAction *closeAction = new QAction(q); closeAction->setText(KMessageWidget::tr("&Close")); closeAction->setToolTip(KMessageWidget::tr("Close message")); - closeAction->setIcon(FilePath::instance()->icon("actions", "dialog-close")); + closeAction->setIcon(FilePath::instance()->icon("actions", "message-close", false)); QObject::connect(closeAction, SIGNAL(triggered(bool)), q, SLOT(animatedHide())); @@ -293,11 +293,7 @@ void KMessageWidget::setMessageType(KMessageWidget::MessageType type) } // Colors -#ifdef Q_OS_MAC fg = palette().light().color(); -#else - fg = palette().highlightedText().color(); -#endif bg0 = bg1.lighter(110); bg2 = bg1.darker(110); border = darkShade(bg1); @@ -311,6 +307,7 @@ void KMessageWidget::setMessageType(KMessageWidget::MessageType type) "border-radius: 5px;" "border: 1px solid %4;" "margin: %5px;" + "padding: 5px;" "}" ".QLabel { color: %6; }" ))