From 5ec00ee7622847f3831659d0b8ee49d248f0ffe2 Mon Sep 17 00:00:00 2001
From: Tyler Goodlet <jgbt@protonmail.com>
Date: Sun, 25 Jul 2021 15:07:02 -0400
Subject: [PATCH] Size view delegate from monkey patched parent

---
 piker/ui/_forms.py | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/piker/ui/_forms.py b/piker/ui/_forms.py
index 6fe419ad..f33b25f1 100644
--- a/piker/ui/_forms.py
+++ b/piker/ui/_forms.py
@@ -108,12 +108,11 @@ class FontAndChartAwareLineEdit(QLineEdit):
 
 
 class FontScaledDelegate(QStyledItemDelegate):
-    """
+    '''
     Super simple view delegate to render text in the same
     font size as the search widget.
 
-    """
-
+    '''
     def __init__(
         self,
 
@@ -136,8 +135,13 @@ class FontScaledDelegate(QStyledItemDelegate):
         # value = index.data()
         # br = self.dpi_font.boundingRect(value)
         # w, h = br.width(), br.height()
-        w, h  = self.parent()._max_item_size
-        return QSize(w, h)
+        parent = self.parent()
+
+        if getattr(parent, '_max_item_size', None):
+            return QSize(*self.parent()._max_item_size)
+
+        else:
+            return super().sizeHint(option, index)
 
 
 class FieldsForm(QtGui.QWidget):
@@ -238,6 +242,7 @@ class FieldsForm(QtGui.QWidget):
             QSizePolicy.Fixed,
         )
         view = select.view()
+        view.setUniformItemSizes(True)
         view.setItemDelegate(FontScaledDelegate(view))
 
         # compute maximum item size so that the weird
@@ -246,12 +251,14 @@ class FieldsForm(QtGui.QWidget):
         values.sort()
         br = _font.boundingRect(str(values[-1]))
         w, h = br.width(), br.height()
+
+        # TODO: something better then this monkey patch
         view._max_item_size = w, h
-        view.setUniformItemSizes(True)
 
         # limit to 6 items?
         view.setMaximumHeight(6*h)
         select.show()
+
         self.hbox.addWidget(select)
 
         return select