From 2f14f58911539fd67228c9113665c5df26061154 Mon Sep 17 00:00:00 2001
From: Vicki Pfau <vi@endrift.com>
Date: Mon, 23 Jan 2017 22:18:14 -0800
Subject: [PATCH] Qt: Merge "Save" and "OK" buttons in shader options

---
 CHANGES                            |  1 +
 src/platform/qt/ShaderSelector.cpp |  3 ++-
 src/platform/qt/ShaderSelector.ui  | 21 ++-------------------
 3 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/CHANGES b/CHANGES
index 5e472c38f..633dff928 100644
--- a/CHANGES
+++ b/CHANGES
@@ -52,6 +52,7 @@ Misc:
  - Feature: Support ImageMagick 7
  - All: Move time.h include to common.h
  - CMake: Add ability to just print version string
+ - Qt: Merge "Save" and "OK" buttons in shader options
 
 0.5.2: (2016-12-31)
 Bugfixes:
diff --git a/src/platform/qt/ShaderSelector.cpp b/src/platform/qt/ShaderSelector.cpp
index 928cef505..4eb743fce 100644
--- a/src/platform/qt/ShaderSelector.cpp
+++ b/src/platform/qt/ShaderSelector.cpp
@@ -264,9 +264,10 @@ void ShaderSelector::buttonPressed(QAbstractButton* button) {
 	case QDialogButtonBox::Reset:
 		emit reset();
 		break;
-	case QDialogButtonBox::Save:
+	case QDialogButtonBox::Ok:
 		m_config->setOption("shader", m_shaderPath);
 		emit saved();
+		close();
 		break;
  	case QDialogButtonBox::RestoreDefaults:
 		emit resetToDefault();
diff --git a/src/platform/qt/ShaderSelector.ui b/src/platform/qt/ShaderSelector.ui
index 1465d92c0..a2af540a2 100644
--- a/src/platform/qt/ShaderSelector.ui
+++ b/src/platform/qt/ShaderSelector.ui
@@ -104,29 +104,12 @@
       <enum>Qt::Horizontal</enum>
      </property>
      <property name="standardButtons">
-      <set>QDialogButtonBox::Ok|QDialogButtonBox::Reset|QDialogButtonBox::RestoreDefaults|QDialogButtonBox::Save</set>
+      <set>QDialogButtonBox::Ok|QDialogButtonBox::Reset|QDialogButtonBox::RestoreDefaults</set>
      </property>
     </widget>
    </item>
   </layout>
  </widget>
  <resources/>
- <connections>
-  <connection>
-   <sender>buttonBox</sender>
-   <signal>accepted()</signal>
-   <receiver>ShaderSelector</receiver>
-   <slot>close()</slot>
-   <hints>
-    <hint type="sourcelabel">
-     <x>248</x>
-     <y>254</y>
-    </hint>
-    <hint type="destinationlabel">
-     <x>157</x>
-     <y>274</y>
-    </hint>
-   </hints>
-  </connection>
- </connections>
+ <connections/>
 </ui>