Browse Source

Fix for showing printer connect button on component load, use newer CuraApplication as target

ChrisTerBeke 7 years ago
parent
commit
2986a17c1d

+ 3 - 4
resources/qml/Cura.qml

@@ -24,7 +24,7 @@ UM.MainWindow
     // It should be phased out in newer plugin versions.
     // It should be phased out in newer plugin versions.
     Connections
     Connections
     {
     {
-        target: Printer
+        target: CuraApplication
         onShowPrintMonitor: {
         onShowPrintMonitor: {
             if (show) {
             if (show) {
                 UM.Controller.setActiveStage("MonitorStage")
                 UM.Controller.setActiveStage("MonitorStage")
@@ -377,7 +377,6 @@ UM.MainWindow
                 }
                 }
 
 
                 width: UM.Theme.getSize("sidebar").width
                 width: UM.Theme.getSize("sidebar").width
-                z: 1
 
 
                 source: UM.Controller.activeStage.sidebarComponent
                 source: UM.Controller.activeStage.sidebarComponent
             }
             }
@@ -819,7 +818,7 @@ UM.MainWindow
 
 
     Connections
     Connections
     {
     {
-        target: Printer
+        target: CuraApplication
         onShowMessageBox:
         onShowMessageBox:
         {
         {
             messageDialog.title = title
             messageDialog.title = title
@@ -865,7 +864,7 @@ UM.MainWindow
 
 
     Connections
     Connections
     {
     {
-        target: Printer
+        target: CuraApplication
         onRequestAddPrinter:
         onRequestAddPrinter:
         {
         {
             addMachineDialog.visible = true
             addMachineDialog.visible = true

+ 12 - 7
resources/qml/MonitorButton.qml

@@ -198,14 +198,19 @@ Item
             spacing: UM.Theme.getSize("default_margin").width
             spacing: UM.Theme.getSize("default_margin").width
         }
         }
 
 
+        Component.onCompleted: {
+            updateAdditionalComponents("monitorButtons")
+        }
+
         Connections {
         Connections {
-            target: Printer
-            onAdditionalComponentsChanged:
-            {
-                if(areaId == "monitorButtons") {
-                    for (var component in CuraApplication.additionalComponents["monitorButtons"]) {
-                        CuraApplication.additionalComponents["monitorButtons"][component].parent = additionalComponentsRow
-                    }
+            target: CuraApplication
+            onAdditionalComponentsChanged: updateAdditionalComponents
+        }
+
+        function updateAdditionalComponents (areaId) {
+            if(areaId == "monitorButtons") {
+                for (var component in CuraApplication.additionalComponents["monitorButtons"]) {
+                    CuraApplication.additionalComponents["monitorButtons"][component].parent = additionalComponentsRow
                 }
                 }
             }
             }
         }
         }

+ 12 - 7
resources/qml/Preferences/MachinesPage.qml

@@ -222,14 +222,19 @@ UM.ManagementPage
             }
             }
         }
         }
 
 
+        Component.onCompleted: {
+            addAdditionalComponents("machinesDetailPane")
+        }
+
         Connections {
         Connections {
-            target: Printer
-            onAdditionalComponentsChanged:
-            {
-                if(areaId == "machinesDetailPane") {
-                    for (var component in CuraApplication.additionalComponents["machinesDetailPane"]) {
-                        CuraApplication.additionalComponents["machinesDetailPane"][component].parent = additionalComponentsColumn
-                    }
+            target: CuraApplication
+            onAdditionalComponentsChanged: addAdditionalComponents
+        }
+
+        function addAdditionalComponents (areaId) {
+            if(areaId == "machinesDetailPane") {
+                for (var component in CuraApplication.additionalComponents["machinesDetailPane"]) {
+                    CuraApplication.additionalComponents["machinesDetailPane"][component].parent = additionalComponentsColumn
                 }
                 }
             }
             }
         }
         }

+ 12 - 7
resources/qml/SaveButton.qml

@@ -135,14 +135,19 @@ Item {
             spacing: UM.Theme.getSize("default_margin").width
             spacing: UM.Theme.getSize("default_margin").width
         }
         }
 
 
+        Component.onCompleted: {
+            addAdditionalComponents("saveButton")
+        }
+
         Connections {
         Connections {
-            target: Printer
-            onAdditionalComponentsChanged:
-            {
-                if(areaId == "saveButton") {
-                    for (var component in CuraApplication.additionalComponents["saveButton"]) {
-                        CuraApplication.additionalComponents["saveButton"][component].parent = additionalComponentsRow
-                    }
+            target: CuraApplication
+            onAdditionalComponentsChanged: addAdditionalComponents
+        }
+
+        function addAdditionalComponents (areaId) {
+            if(areaId == "saveButton") {
+                for (var component in CuraApplication.additionalComponents["saveButton"]) {
+                    CuraApplication.additionalComponents["saveButton"][component].parent = additionalComponentsRow
                 }
                 }
             }
             }
         }
         }

+ 0 - 1
resources/qml/Sidebar.qml

@@ -549,7 +549,6 @@ Rectangle
         visible: monitoringPrint
         visible: monitoringPrint
     }
     }
 
 
-
     SidebarTooltip
     SidebarTooltip
     {
     {
         id: tooltip;
         id: tooltip;