Fixed bug in saving node widget enabled state

Author: Eloy Retamino

include/QtNodes/internal/AbstractNodeGeometry.hpp

@@ -74,13 +74,14 @@ class NODE_EDITOR_PUBLIC AbstractNodeGeometry
 
     virtual QRect enableWidgetHandleRect(NodeId const nodeId) const = 0;
 
-    void setEnableEmbeddedWidget(bool val)
-    { _embeddedWidgetEnabled = val; }
+    void setEnableEmbeddedWidget(NodeId const nodeId, bool val)
+    { _embeddedWidgetEnabled[nodeId] = val; }
 
 protected:
+
     AbstractGraphModel &_graphModel;
 
-    bool _embeddedWidgetEnabled = true;
+    std::map<NodeId, bool> _embeddedWidgetEnabled;
 
 };
 

src/DefaultHorizontalNodeGeometry.cpp

@@ -33,7 +33,7 @@ void DefaultHorizontalNodeGeometry::recomputeSize(NodeId const nodeId) const
     unsigned int height = maxVerticalPortsExtent(nodeId);
 
     auto w = _graphModel.nodeData<QWidget *>(nodeId, NodeRole::Widget);
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         height = std::max(height, static_cast<unsigned int>(w->height()));
     }
 
@@ -50,7 +50,7 @@ void DefaultHorizontalNodeGeometry::recomputeSize(NodeId const nodeId) const
 
     unsigned int width = inPortWidth + outPortWidth + 4 * _portSpasing;
 
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         width += w->width();
     }
 
@@ -153,7 +153,7 @@ QPointF DefaultHorizontalNodeGeometry::widgetPosition(NodeId const nodeId) const
     unsigned int captionHeight = captionRect(nodeId).height();
 
     auto w = _graphModel.nodeData<QWidget *>(nodeId, NodeRole::Widget);
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         // If the widget wants to use as much vertical space as possible,
         // place it immediately after the caption.
         if (w->sizePolicy().verticalPolicy() & QSizePolicy::ExpandFlag) {

src/DefaultVerticalNodeGeometry.cpp

@@ -33,7 +33,7 @@ void DefaultVerticalNodeGeometry::recomputeSize(NodeId const nodeId) const
     unsigned int height = _portSpasing; // maxHorizontalPortsExtent(nodeId);
 
     auto w = _graphModel.nodeData<QWidget *>(nodeId, NodeRole::Widget);
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         height = std::max(height, static_cast<unsigned int>(w->height()));
     }
 
@@ -65,7 +65,7 @@ void DefaultVerticalNodeGeometry::recomputeSize(NodeId const nodeId) const
 
     unsigned int width = std::max(totalInPortsWidth, totalOutPortsWidth);
 
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         width = std::max(width, static_cast<unsigned int>(w->width()));
     }
 
@@ -180,7 +180,7 @@ QPointF DefaultVerticalNodeGeometry::widgetPosition(NodeId const nodeId) const
     unsigned int captionHeight = captionRect(nodeId).height();
 
     auto w = _graphModel.nodeData<QWidget *>(nodeId, NodeRole::Widget);
-    if (_embeddedWidgetEnabled && w) {
+    if (_embeddedWidgetEnabled.count(nodeId) && _embeddedWidgetEnabled.at(nodeId) && w) {
         // If the widget wants to use as much vertical space as possible,
         // place it immediately after the caption.
         if (w->sizePolicy().verticalPolicy() & QSizePolicy::ExpandFlag) {