Changeset fa762f4 in sasview for src/sas/qtgui/Utilities/GridPanel.py

Timestamp:

Sep 25, 2018 7:06:48 AM (6 years ago)

Author:

Piotr Rozyczko <piotr.rozyczko@…>

Branches:

ESS_GUI, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc

Children:

624c8747

Parents:

9b9ec10

git-author:

Piotr Rozyczko <piotr.rozyczko@…> (09/25/18 07:04:07)

git-committer:

Piotr Rozyczko <piotr.rozyczko@…> (09/25/18 07:06:48)

Message:

Fixed lifecycle/visibility of the Grid Viewer SASVIEW-1122

File:

• src/sas/qtgui/Utilities/GridPanel.py (modified) (4 diffs)

src/sas/qtgui/Utilities/GridPanel.py

@@ -18 +18 @@
     ERROR_COLUMN_CAPTION = " (Err)"
     IS_WIN = (sys.platform == 'win32')
+    windowClosedSignal = QtCore.pyqtSignal()
     def __init__(self, parent = None, output_data=None):
 
@@ -54 +55 @@
         # Fill in the table from input data
         self.setupTable(widget=self.tblParams, data=output_data)
-        #TODO: This is not what was inteded to be.
         if output_data is not None:
             # Set a table tooltip describing the model
@@ -64 +64 @@
         Overwrite QDialog close method to allow for custom widget close
         """
-        # Maybe we should just minimize
-        self.setWindowState(QtCore.Qt.WindowMinimized)
+        # notify the parent so it hides this window
+        self.windowClosedSignal.emit()
         event.ignore()
 
@@ -153 +153 @@
         Create a new tab with batch fitting results
         """
-        self.addTabPage()
+        if self.has_data:
+            self.addTabPage()
         # Update the new widget
         # Fill in the table from input data in the last/newest page