From b6a00b9f7f3cc753103c6546ea21bc819f17be28 Mon Sep 17 00:00:00 2001 From: ChrisTerBeke Date: Tue, 9 Jul 2019 08:26:44 +0200 Subject: [PATCH] Hide 'move to top' opion from actions --- .../resources/qml/MonitorContextMenu.qml | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/plugins/UM3NetworkPrinting/resources/qml/MonitorContextMenu.qml b/plugins/UM3NetworkPrinting/resources/qml/MonitorContextMenu.qml index fbae66117e..5a40aef780 100644 --- a/plugins/UM3NetworkPrinting/resources/qml/MonitorContextMenu.qml +++ b/plugins/UM3NetworkPrinting/resources/qml/MonitorContextMenu.qml @@ -46,6 +46,9 @@ Item spacing: Math.floor(UM.Theme.getSize("default_margin").height / 2) + // Due to an issue with the ordering if print jobs caused by the Qt list models, + // we hide the 'move to top' feature for now as it's not displayed on the appropriate elements. + // Solving the ordering issue will cost more time than we currently have available. PrintJobContextMenuItem { onClicked: { sendToTopConfirmationDialog.visible = true; @@ -53,11 +56,11 @@ Item } text: catalog.i18nc("@label", "Move to top"); visible: { - if (printJob && (printJob.state == "queued" || printJob.state == "error") && !isAssigned(printJob)) { - if (OutputDevice && OutputDevice.queuedPrintJobs[0]) { - return OutputDevice.queuedPrintJobs[0].key != printJob.key; - } - } +// if (printJob && (printJob.state == "queued" || printJob.state == "error") && !isAssigned(printJob)) { +// if (OutputDevice && OutputDevice.queuedPrintJobs[0]) { +// return OutputDevice.queuedPrintJobs[0].key != printJob.key; +// } +// } return false; } }