Move the visible check within the component.

Contributes to CURA-5772.
This commit is contained in:
Diego Prado Gesto 2018-10-26 16:31:03 +02:00
parent 5a5adb71cd
commit c8bdf7321c
2 changed files with 3 additions and 4 deletions

View File

@ -13,9 +13,6 @@ Item
signal showTooltip(Item item, point location, string text) signal showTooltip(Item item, point location, string text)
signal hideTooltip() signal hideTooltip()
property bool isNetworkPrinter: Cura.MachineManager.activeMachineNetworkKey != ""
property bool printerConnected: Cura.MachineManager.printerConnected
UM.I18nCatalog UM.I18nCatalog
{ {
id: catalog id: catalog
@ -53,7 +50,6 @@ Item
Cura.QuickConfigurationSelector Cura.QuickConfigurationSelector
{ {
id: configSelection id: configSelection
visible: isNetworkPrinter && printerConnected
width: visible ? Math.round(UM.Theme.getSize("sidebar").width * 0.15) : 0 width: visible ? Math.round(UM.Theme.getSize("sidebar").width * 0.15) : 0
panelWidth: Math.round(0.8 * UM.Theme.getSize("sidebar").width) panelWidth: Math.round(0.8 * UM.Theme.getSize("sidebar").width)
height: prepareMenu.height height: prepareMenu.height

View File

@ -14,6 +14,9 @@ Item
property var connectedDevice: Cura.MachineManager.printerOutputDevices.length >= 1 ? Cura.MachineManager.printerOutputDevices[0] : null property var connectedDevice: Cura.MachineManager.printerOutputDevices.length >= 1 ? Cura.MachineManager.printerOutputDevices[0] : null
property var panelWidth: control.width property var panelWidth: control.width
// Make this component only visible when it's a network printer and it is connected
visible: Cura.MachineManager.activeMachineNetworkKey != "" && Cura.MachineManager.printerConnected
function switchPopupState() function switchPopupState()
{ {
popup.visible ? popup.close() : popup.open() popup.visible ? popup.close() : popup.open()