Address review

1. Drop dead prepareGeometryChange()/update() calls in
setPotential(), split unrelated terminaleditor.h re-indent into its
own change

2. Removed: setPotential() had prepareGeometryChange()/update() left over from
the label-display mechanism that was dropped before this PR
This commit is contained in:
Levi Jetzer
2026-09-04 13:12:21 +02:00
parent e410896f5d
commit 1e1039dd32
2 changed files with 7 additions and 9 deletions
@@ -479,9 +479,7 @@ void PartTerminal::setMasterLabelIndex(int index)
void PartTerminal::setPotential(const QString &potential)
{
if (d->m_potential == potential) return;
prepareGeometryChange();
d->m_potential = potential;
update();
emit potentialChanged();
}
+7 -7
View File
@@ -65,13 +65,13 @@ class TerminalEditor : public ElementItemEditor
void labelSizeEdited();
void labelRotationEdited();
void labelAlignClicked();
void labelFrameEdited();
void labelColorClicked();
void activeConnections(bool active);
void activeChangeConnections(bool active);
void useMasterLabelEdited();
void masterLabelIndexEdited();
bool updateMasterLabelVisibility();
void labelFrameEdited();
void labelColorClicked();
void activeConnections(bool active);
void activeChangeConnections(bool active);
void useMasterLabelEdited();
void masterLabelIndexEdited();
bool updateMasterLabelVisibility();
private:
Ui::TerminalEditor *ui;