bugfix: Fix issue labscript-suite#61, shuffle checkstate not preserved upon removing and adding 'outer' globals.

Also fix that the checkox is initially set to be a tristate one. It should not
be settable to a partially checked state by the user.

Author: chrisjbillington

__main__.py

@@ -2031,13 +2031,19 @@ def update_global_shuffle_state(self, *args, **kwargs):
         if not all_checked and not none_checked:
             self.ui.pushButton_shuffle.setTristate(True)
             self.ui.pushButton_shuffle.setCheckState(QtCore.Qt.PartiallyChecked)
-        elif none_checked:
+        elif none_checked and not all_checked:
             self.ui.pushButton_shuffle.setTristate(False)
             self.ui.pushButton_shuffle.setCheckState(QtCore.Qt.Unchecked)
-        else:
+        elif all_checked and not none_checked:
             self.ui.pushButton_shuffle.setTristate(False)
             self.ui.pushButton_shuffle.setCheckState(QtCore.Qt.Checked)
-    
+        else:
+            # No axes. Clear if partially checked, otherwise else leave it alone:
+            if self.ui.pushButton_shuffle.checkState() == QtCore.Qt.PartiallyChecked:
+                self.ui.pushButton_shuffle.setTristate(False)
+                self.ui.pushButton_shuffle.setCheckState(QtCore.Qt.Unchecked)
+
+
     def on_master_shuffle_clicked(self, state):
         if state in [QtCore.Qt.Checked, QtCore.Qt.Unchecked]:
             self.ui.pushButton_shuffle.setTristate(False)

main.ui

@@ -229,7 +229,7 @@ subprocess</string>
             <bool>true</bool>
            </property>
            <property name="tristate">
-            <bool>true</bool>
+            <bool>false</bool>
            </property>
           </widget>
          </item>