Usability fixes

Make the edit area disabled until the user selects he wants custom
positions.
Its a source of confusion if the user can edit the positions but doesn't
see anything happen in his decorations upon apply.  This gives immediate
feedback that the checkbox should be enabled.

Conflicts:

	kwin/kcmkwin/kwindecoration/buttonsconfigdialog.cpp

kcmkwin/kwindecoration/buttons.ui

@@ -11,6 +11,9 @@
    </rect>
   </property>
   <layout class="QVBoxLayout" name="verticalLayout">
+   <property name="margin">
+    <number>0</number>
+   </property>
    <item>
     <widget class="QCheckBox" name="showToolTipsCheckBox">
      <property name="whatsThis">
@@ -32,7 +35,11 @@
     </widget>
    </item>
    <item>
-    <widget class="KWin::ButtonPositionWidget" name="buttonPositionWidget" native="true"/>
+    <widget class="KWin::ButtonPositionWidget" name="buttonPositionWidget" native="true">
+     <property name="enabled">
+      <bool>false</bool>
+     </property>
+    </widget>
    </item>
   </layout>
  </widget>
@@ -45,5 +52,22 @@
   </customwidget>
  </customwidgets>
  <resources/>
- <connections/>
+ <connections>
+  <connection>
+   <sender>useCustomButtonPositionsCheckBox</sender>
+   <signal>clicked(bool)</signal>
+   <receiver>buttonPositionWidget</receiver>
+   <slot>setEnabled(bool)</slot>
+   <hints>
+    <hint type="sourcelabel">
+     <x>205</x>
+     <y>39</y>
+    </hint>
+    <hint type="destinationlabel">
+     <x>231</x>
+     <y>90</y>
+    </hint>
+   </hints>
+  </connection>
+ </connections>
 </ui>

kcmkwin/kwindecoration/buttonsconfigdialog.cpp

@@ -45,7 +45,7 @@ KWinDecorationButtonsConfigDialog::KWinDecorationButtonsConfigDialog(bool custom
     setButtons(KDialog::Ok | KDialog::Cancel | KDialog::Default | KDialog::Reset);
     enableButton(KDialog::Reset, false);
     QVBoxLayout* layout = new QVBoxLayout;
-    layout->addWidget(m_ui);
+    m_ui->buttonPositionWidget->setEnabled(customPositions);
 
     QWidget* main = new QWidget(this);
     main->setLayout(layout);