Browse Source

Hide 'move to top' opion from actions

ChrisTerBeke 5 years ago
parent
commit
b6a00b9f7f
1 changed files with 8 additions and 5 deletions
  1. 8 5
      plugins/UM3NetworkPrinting/resources/qml/MonitorContextMenu.qml

+ 8 - 5
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;
                     }
                 }