Skip to content

Commit 56ff23a

Browse files
committed
fix: use QLayout.SetFixedSize to auto-resize dialog on checkbox toggle
1 parent b3ed570 commit 56ff23a

1 file changed

Lines changed: 5 additions & 20 deletions

File tree

src/settings_dialog.py

Lines changed: 5 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
QGroupBox,
2121
QHBoxLayout,
2222
QLabel,
23+
QLayout,
2324
QLineEdit,
2425
QMessageBox,
2526
QPlainTextEdit,
@@ -93,6 +94,7 @@ def __init__(
9394
self._working = copy.deepcopy(config)
9495

9596
self._build_ui()
97+
self.layout().setSizeConstraint(QLayout.SetFixedSize)
9698
self._populate(self._working)
9799

98100
# ------------------------------------------------------------------
@@ -194,9 +196,7 @@ def _build_stt_tab(self) -> None:
194196
)
195197

196198
self._stt_fb_group.setVisible(False)
197-
self._stt_fb_check.toggled.connect(
198-
lambda checked: self._set_dynamic_group_visible(self._stt_fb_group, checked)
199-
)
199+
self._stt_fb_check.toggled.connect(self._stt_fb_group.setVisible)
200200
form.addRow(self._stt_fb_group)
201201

202202
self._tabs.addTab(tab, "STT")
@@ -240,15 +240,11 @@ def _build_llm_tab(self) -> None:
240240
)
241241

242242
self._llm_fb_group.setVisible(False)
243-
self._llm_fb_check.toggled.connect(
244-
lambda checked: self._set_dynamic_group_visible(self._llm_fb_group, checked)
245-
)
243+
self._llm_fb_check.toggled.connect(self._llm_fb_group.setVisible)
246244
llm_form.addRow(self._llm_fb_group)
247245

248246
self._llm_group.setVisible(False)
249-
self._llm_check.toggled.connect(
250-
lambda checked: self._set_dynamic_group_visible(self._llm_group, checked)
251-
)
247+
self._llm_check.toggled.connect(self._llm_group.setVisible)
252248
form.addRow(self._llm_group)
253249

254250
self._tabs.addTab(tab, "LLM")
@@ -410,17 +406,6 @@ def _on_calibrate(self) -> None:
410406
except Exception as e:
411407
QMessageBox.warning(self, "Calibration Failed", str(e))
412408

413-
# ------------------------------------------------------------------
414-
# Dynamic reveals
415-
# ------------------------------------------------------------------
416-
417-
def _set_dynamic_group_visible(self, group: QWidget, visible: bool) -> None:
418-
group.setVisible(visible)
419-
420-
if self.layout() is not None:
421-
self.layout().activate()
422-
self.resize(self.sizeHint())
423-
424409
# ------------------------------------------------------------------
425410
# Validation
426411
# ------------------------------------------------------------------

0 commit comments

Comments
 (0)