Просмотр исходного кода

Fix applying configuration when clicking

Also no longer use that signal for it. It was completely unnecessary.

Contributes to issue CURA-5876.
Ghostkeeper 6 лет назад
Родитель
Сommit
b15c272d23

+ 5 - 1
resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml

@@ -12,7 +12,6 @@ Button
     id: configurationItem
 
     property var configuration: null
-    signal activateConfiguration()
 
     height: childrenRect.height
 
@@ -111,4 +110,9 @@ Button
             configurationItem.checked = Cura.MachineManager.matchesConfiguration(configuration)
         }
     }
+
+    onClicked:
+    {
+        Cura.MachineManager.applyRemoteConfiguration(configuration)
+    }
 }

+ 0 - 5
resources/qml/Menus/ConfigurationMenu/ConfigurationListView.qml

@@ -70,11 +70,6 @@ Column
             {
                 width: parent.width - UM.Theme.getSize("default_margin").width
                 configuration: modelData
-                onActivateConfiguration:
-                {
-                    switchPopupState()
-                    Cura.MachineManager.applyRemoteConfiguration(configuration)
-                }
             }
         }
     }