Add 3D border optionfeat/kicker-3d-border

Kicker does not draw a border by default, resulting in a flat look.
Only under specific conditions (e.g. a panel background) a little 1px border
gets drawn above the panel to improve its visibility.

This commit adds the option to manually enable a 3D border around the panel.
This makes the panel better stand out from the background and fits better
with classic styles.

Signed-off-by: Philippe Mavridis <philippe.mavridis@yandex.com>

2 files changed, 20 insertions, 1 deletions

diff --git a/kcontrol/kicker/advancedDialog.cpp b/kcontrol/kicker/advancedDialog.cpp
index e9ed08b0a..ef163ae28 100644
--- a/kcontrol/kicker/advancedDialog.cpp
+++ b/kcontrol/kicker/advancedDialog.cpp
@@ -69,6 +69,8 @@ advancedDialog::advancedDialog(TQWidget* parent, const char* name)
             this, TQ_SLOT(changed()));
     connect(m_advancedWidget->kickerDeepButtons, TQ_SIGNAL(clicked()),
             this, TQ_SLOT(changed()));
+    connect(m_advancedWidget->kickerBorder, TQ_SIGNAL(clicked()),
+            this, TQ_SLOT(changed()));
     load();
 }
 
@@ -117,6 +119,8 @@ void advancedDialog::load()
     m_advancedWidget->kickerResizeHandle->setChecked( useKickerResizeHandle );
     bool usekickerDeepButtons = c.readBoolEntry("ShowDeepButtons", false);
     m_advancedWidget->kickerDeepButtons->setChecked( usekickerDeepButtons );
+    bool usekickerBorder = c.readBoolEntry("ShowPanelBorder", false);
+    m_advancedWidget->kickerBorder->setChecked( usekickerBorder );
 
     enableButtonApply(false);
 }
@@ -144,6 +148,8 @@ void advancedDialog::save()
                  m_advancedWidget->kickerResizeHandle->isChecked());
     c.writeEntry("ShowDeepButtons",
                  m_advancedWidget->kickerDeepButtons->isChecked());
+    c.writeEntry("ShowPanelBorder",
+                 m_advancedWidget->kickerBorder->isChecked());
 
     TQStringList elist = c.readListEntry("Extensions2");
     for (TQStringList::Iterator it = elist.begin(); it != elist.end(); ++it)
@@ -180,6 +186,8 @@ void advancedDialog::save()
                              m_advancedWidget->kickerResizeHandle->isChecked());
         extConfig.writeEntry("ShowDeepButtons",
                              m_advancedWidget->kickerDeepButtons->isChecked());
+        extConfig.writeEntry("ShowPanelBorder",
+                             m_advancedWidget->kickerBorder->isChecked());
 
         extConfig.sync();
     }
diff --git a/kcontrol/kicker/advancedOptions.ui b/kcontrol/kicker/advancedOptions.ui
index 0e65401db..ddedd1eaa 100644
--- a/kcontrol/kicker/advancedOptions.ui
+++ b/kcontrol/kicker/advancedOptions.ui
@@ -495,7 +495,18 @@
                         <string>Enabling this option will show a resize handle on the resizable end of each panel.</string>
                     </property>
                 </widget>
-                <widget class="TQCheckBox" row="0" column="1">
+                <widget class="TQCheckBox" row="1" column="0">
+                    <property name="name">
+                        <cstring>kickerBorder</cstring>
+                    </property>
+                    <property name="text">
+                        <string>Show panel border</string>
+                    </property>
+                    <property name="whatsThis" stdset="0">
+                        <string>Enabling this option will draw a 3D border around the panel. Use this for a more classic look.</string>
+                    </property>
+                </widget>
+                <widget class="TQCheckBox" row="2" column="0">
                     <property name="name">
                         <cstring>kickerDeepButtons</cstring>
                     </property>