From 0548f6e3ec42af8cea66489e2349e02cbe8c9f86 Mon Sep 17 00:00:00 2001 From: rob Date: Sat, 17 Jan 2026 14:24:24 -0400 Subject: [PATCH] Let table cells manage widget sizes naturally - Remove fixed heights, let Qt size widgets to fit cells - Increase row height to 40px for more room - Use padding instead of fixed size on remove button Co-Authored-By: Claude Opus 4.5 --- src/cmdforge/gui/dialogs/test_step_dialog.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/cmdforge/gui/dialogs/test_step_dialog.py b/src/cmdforge/gui/dialogs/test_step_dialog.py index 83225c5..4317e2b 100644 --- a/src/cmdforge/gui/dialogs/test_step_dialog.py +++ b/src/cmdforge/gui/dialogs/test_step_dialog.py @@ -299,27 +299,24 @@ class TestStepDialog(QDialog): self.assertions_table.insertRow(row) # Set proper row height for widgets - self.assertions_table.setRowHeight(row, 32) + self.assertions_table.setRowHeight(row, 40) # Type dropdown type_combo = QComboBox() - type_combo.setFixedHeight(26) type_combo.setMinimumWidth(120) + self.assertions_table.setCellWidget(row, 0, type_combo) for type_id, display_name, tooltip in self.ASSERTION_TYPES: type_combo.addItem(display_name, type_id) idx = type_combo.count() - 1 type_combo.setItemData(idx, tooltip, Qt.ToolTipRole) - self.assertions_table.setCellWidget(row, 0, type_combo) # Value input value_edit = QLineEdit() value_edit.setPlaceholderText("Expected value (if applicable)") - value_edit.setFixedHeight(26) self.assertions_table.setCellWidget(row, 1, value_edit) # Remove button - red X for clear delete action btn_remove = QPushButton("✕") - btn_remove.setFixedSize(26, 26) btn_remove.setToolTip("Remove this assertion") btn_remove.setStyleSheet(""" QPushButton { @@ -328,7 +325,8 @@ class TestStepDialog(QDialog): border: 1px solid #fc8181; border-radius: 4px; font-weight: bold; - font-size: 14px; + font-size: 16px; + padding: 4px 8px; } QPushButton:hover { background-color: #feb2b2;