From a987899e5ccca578d0c783379c94564725a9c881 Mon Sep 17 00:00:00 2001 From: itsmattkc Date: Sun, 16 Feb 2020 12:37:10 +1100 Subject: [PATCH] nodeparamviewwidgetbridge: connect checkbox "clicked" signal rather than "toggled" Only signal a change if the user made one. Toggled would be signalled even when we set it programmatically (e.g. by simply creating the checkbox widget and setting its state) causing unnecessary re-caches of the sequence. --- app/widget/nodeparamview/nodeparamviewwidgetbridge.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/widget/nodeparamview/nodeparamviewwidgetbridge.cpp b/app/widget/nodeparamview/nodeparamviewwidgetbridge.cpp index 00e988242..444140f1e 100644 --- a/app/widget/nodeparamview/nodeparamviewwidgetbridge.cpp +++ b/app/widget/nodeparamview/nodeparamviewwidgetbridge.cpp @@ -228,7 +228,7 @@ void NodeParamViewWidgetBridge::CreateWidgets() { QCheckBox* check_box = new QCheckBox(); widgets_.append(check_box); - connect(check_box, &QCheckBox::toggled, this, &NodeParamViewWidgetBridge::WidgetCallback); + connect(check_box, &QCheckBox::clicked, this, &NodeParamViewWidgetBridge::WidgetCallback); break; } case NodeParam::kFont: