From aa25e0746c814b37ab5f9f4e539febb74913855b Mon Sep 17 00:00:00 2001 From: "j.delarago" Date: Mon, 7 Mar 2022 14:35:09 +0100 Subject: [PATCH 01/24] Make burger invisible when no printer is selected in Printer page CURA-9007 --- resources/qml/Preferences/MachinesPage.qml | 1 + 1 file changed, 1 insertion(+) diff --git a/resources/qml/Preferences/MachinesPage.qml b/resources/qml/Preferences/MachinesPage.qml index b1a5d7f19b..28568d041b 100644 --- a/resources/qml/Preferences/MachinesPage.qml +++ b/resources/qml/Preferences/MachinesPage.qml @@ -21,6 +21,7 @@ UM.ManagementPage activeId: Cura.MachineManager.activeMachine !== null ? Cura.MachineManager.activeMachine.id: "" activeIndex: activeMachineIndex() onHamburgeButtonClicked: menu.popup(content_item, content_item.width - menu.width, hamburger_button.height) + hamburgerButtonVisible: Cura.MachineManager.activeMachine !== null function activeMachineIndex() { From 1eb887010a0bf7e69c0659017e965a436c37141e Mon Sep 17 00:00:00 2001 From: "j.delarago" Date: Mon, 7 Mar 2022 15:00:51 +0100 Subject: [PATCH 02/24] Swap TextField for Cura.TextField so it has the same styling as the rest of text fields. CURA-9010 --- resources/qml/JobSpecs.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/JobSpecs.qml b/resources/qml/JobSpecs.qml index 03978cb69e..dbd3fe3c46 100644 --- a/resources/qml/JobSpecs.qml +++ b/resources/qml/JobSpecs.qml @@ -58,7 +58,7 @@ Item } } - TextField + Cura.TextField { id: printJobTextfield anchors.left: printJobPencilIcon.right From 37a4986406e0d9e6b0de800fd764ee6ef034147b Mon Sep 17 00:00:00 2001 From: "j.delarago" Date: Mon, 7 Mar 2022 16:17:03 +0100 Subject: [PATCH 03/24] Reduce width of preferences dialog page selection buttons. Reduce width of profiles list and printers list. CURA-9008 --- resources/themes/cura-light/theme.json | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/resources/themes/cura-light/theme.json b/resources/themes/cura-light/theme.json index 4d89c37609..877f0a1f91 100644 --- a/resources/themes/cura-light/theme.json +++ b/resources/themes/cura-light/theme.json @@ -637,6 +637,8 @@ "banner_icon_size": [2.0, 2.0], - "marketplace_large_icon": [4.0, 4.0] + "marketplace_large_icon": [4.0, 4.0], + + "preferences_page_list": [8.0, 2.0] } } From a49aa6502cee8c25aa3bb8614266058e8162ff9b Mon Sep 17 00:00:00 2001 From: "j.delarago" Date: Mon, 7 Mar 2022 16:56:26 +0100 Subject: [PATCH 04/24] Set material and nozzle selection menus to appear at bottom of button like drop downs. CURA-9004 --- .../qml/Menus/ConfigurationMenu/CustomConfiguration.qml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml b/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml index f3cef1a819..706430988d 100644 --- a/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml +++ b/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml @@ -285,10 +285,11 @@ Item Cura.MaterialMenu { id: materialsMenu + width: materialSelection.width extruderIndex: Cura.ExtruderManager.activeExtruderIndex updateModels: materialSelection.visible } - onClicked: materialsMenu.popup() + onClicked: materialsMenu.popup(0, height) } Item @@ -341,8 +342,9 @@ Item { id: nozzlesMenu extruderIndex: Cura.ExtruderManager.activeExtruderIndex + width: variantSelection.width } - onClicked: nozzlesMenu.popup() + onClicked: nozzlesMenu.popup(0, height) } } From 59158db34b2c02ed3a43c36650fbb2db1c996ea8 Mon Sep 17 00:00:00 2001 From: "j.delarago" Date: Mon, 7 Mar 2022 17:07:14 +0100 Subject: [PATCH 05/24] Lining at the bottom of UnderlineBackground and at the top of the menu give a double width lining. Moved the menu up slightly so lining of UnderlineBackground is hidden. CURA-9004 --- resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml b/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml index 706430988d..4b2b84d39e 100644 --- a/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml +++ b/resources/qml/Menus/ConfigurationMenu/CustomConfiguration.qml @@ -289,7 +289,7 @@ Item extruderIndex: Cura.ExtruderManager.activeExtruderIndex updateModels: materialSelection.visible } - onClicked: materialsMenu.popup(0, height) + onClicked: materialsMenu.popup(0, height - UM.Theme.getSize("default_lining").height) } Item @@ -344,7 +344,7 @@ Item extruderIndex: Cura.ExtruderManager.activeExtruderIndex width: variantSelection.width } - onClicked: nozzlesMenu.popup(0, height) + onClicked: nozzlesMenu.popup(0, height - UM.Theme.getSize("default_lining").height) } } From 912db08b234382b1cc9280f9629e715512663529 Mon Sep 17 00:00:00 2001 From: casper Date: Tue, 8 Mar 2022 09:11:49 +0100 Subject: [PATCH 06/24] Let height of preferences_page_list themes value set height of list item CURA-9008 --- resources/themes/cura-light/theme.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/themes/cura-light/theme.json b/resources/themes/cura-light/theme.json index 877f0a1f91..2f14a5be78 100644 --- a/resources/themes/cura-light/theme.json +++ b/resources/themes/cura-light/theme.json @@ -639,6 +639,6 @@ "marketplace_large_icon": [4.0, 4.0], - "preferences_page_list": [8.0, 2.0] + "preferences_page_list_item": [8.0, 2.0] } } From a9a0d4280a00deb66f2544e041ab379d7f9eaba4 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Tue, 8 Mar 2022 14:27:00 +0100 Subject: [PATCH 07/24] Fix Jobspecs text alignment CURA-9010 --- resources/qml/JobSpecs.qml | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/resources/qml/JobSpecs.qml b/resources/qml/JobSpecs.qml index dbd3fe3c46..102496d02e 100644 --- a/resources/qml/JobSpecs.qml +++ b/resources/qml/JobSpecs.qml @@ -68,6 +68,14 @@ Item maximumLength: 120 text: PrintInformation === null ? "" : PrintInformation.jobName horizontalAlignment: TextInput.AlignLeft + onTextChanged: + { + if (!activeFocus) + { + // Text is changed from outside, reset the cursor position. + cursorPosition = 0 + } + } property string textBeforeEdit: "" @@ -86,12 +94,12 @@ Item PrintInformation.setJobName(new_name, true) } printJobTextfield.focus = false + cursorPosition = 0 } validator: RegExpValidator { regExp: /^[^\\\/\*\?\|\[\]]*$/ } - font: UM.Theme.getFont("default") color: UM.Theme.getColor("text_scene") background: Item {} selectByMouse: true From 0e2848796c4f85a826bbcdd8e96c0c4504f74454 Mon Sep 17 00:00:00 2001 From: casper Date: Wed, 9 Mar 2022 09:46:57 +0100 Subject: [PATCH 08/24] Increase text field width to prevent inner text from shifting CURA-9010 --- resources/qml/JobSpecs.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/JobSpecs.qml b/resources/qml/JobSpecs.qml index 102496d02e..1a5f40466a 100644 --- a/resources/qml/JobSpecs.qml +++ b/resources/qml/JobSpecs.qml @@ -64,7 +64,7 @@ Item anchors.left: printJobPencilIcon.right anchors.leftMargin: UM.Theme.getSize("narrow_margin").width height: UM.Theme.getSize("jobspecs_line").height - width: Math.max(contentWidth + UM.Theme.getSize("default_margin").width, 50) + width: Math.max(contentWidth + UM.Theme.getSize("default_margin").width + 2, 50) // add two pixels to width to prevent inner text from shifting maximumLength: 120 text: PrintInformation === null ? "" : PrintInformation.jobName horizontalAlignment: TextInput.AlignLeft From 70f66648f4cbf6295e76f7268fe390d1da30647a Mon Sep 17 00:00:00 2001 From: casper Date: Wed, 9 Mar 2022 12:10:49 +0100 Subject: [PATCH 09/24] Recolor placeholder icon for packages/plugins correctly in dark-mode As we don't want to recolor the icon if the package has a custom icon a `Image` or `UM.RecolorImage` is shown conditionally. CURA-8975 --- .../resources/qml/PackageCardHeader.qml | 41 +++++++++++++------ 1 file changed, 29 insertions(+), 12 deletions(-) diff --git a/plugins/Marketplace/resources/qml/PackageCardHeader.qml b/plugins/Marketplace/resources/qml/PackageCardHeader.qml index 013704793f..1c39d4b063 100644 --- a/plugins/Marketplace/resources/qml/PackageCardHeader.qml +++ b/plugins/Marketplace/resources/qml/PackageCardHeader.qml @@ -24,7 +24,7 @@ Item height: UM.Theme.getSize("card").height // card icon - Image + Item { id: packageItem anchors @@ -35,19 +35,36 @@ Item } width: UM.Theme.getSize("card_icon").width height: width - sourceSize.height: height - sourceSize.width: width - source: + + property bool packageHasIcon: packageData.iconUrl != "" + + Image { - if (packageData.iconUrl != "") + visible: parent.packageHasIcon + anchors.fill: parent + source: packageData.iconUrl + sourceSize.height: height + sourceSize.width: width + } + + UM.RecolorImage + { + visible: !parent.packageHasIcon + anchors.fill: parent + sourceSize.height: height + sourceSize.width: width + color: UM.Theme.getColor("text") + source: { - return packageData.iconUrl - } - switch (packageData.packageType) - { - case "plugin": return "../images/Plugin.svg"; - case "material": return "../images/Spool.svg"; - default: return "../images/placeholder.svg"; + switch (packageData.packageType) + { + case "plugin": + return "../images/Plugin.svg"; + case "material": + return "../images/Spool.svg"; + default: + return "../images/placeholder.svg"; + } } } } From fd3ef2378465a6fbf9f4bb365045de20fb89790e Mon Sep 17 00:00:00 2001 From: casper Date: Wed, 9 Mar 2022 12:23:21 +0100 Subject: [PATCH 10/24] Swap buttons in `MonitorConfigOverrideDialog` for cura buttons CURA-8975 --- .../qml/MonitorConfigOverrideDialog.qml | 24 +++++++++---------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/plugins/UM3NetworkPrinting/resources/qml/MonitorConfigOverrideDialog.qml b/plugins/UM3NetworkPrinting/resources/qml/MonitorConfigOverrideDialog.qml index 343e3f79e1..47586f3925 100644 --- a/plugins/UM3NetworkPrinting/resources/qml/MonitorConfigOverrideDialog.qml +++ b/plugins/UM3NetworkPrinting/resources/qml/MonitorConfigOverrideDialog.qml @@ -5,6 +5,7 @@ import QtQuick 2.3 import QtQuick.Controls 2.4 import QtQuick.Layouts 1.3 import UM 1.5 as UM +import Cura 1.5 as Cura UM.Dialog { @@ -17,12 +18,21 @@ UM.Dialog width: minimumWidth height: minimumHeight title: catalog.i18nc("@title:window", "Configuration Changes") + buttonSpacing: UM.Theme.getSize("narrow_margin").width rightButtons: [ - Button + Cura.TertiaryButton + { + id: cancelButton + text: catalog.i18nc("@action:button", "Cancel") + onClicked: + { + overrideConfirmationDialog.reject() + } + }, + Cura.PrimaryButton { id: overrideButton - anchors.margins: UM.Theme.getSize("default_margin").width text: catalog.i18nc("@action:button", "Override") onClicked: { @@ -49,16 +59,6 @@ UM.Dialog } return true } - }, - Button - { - id: cancelButton - anchors.margins: UM.Theme.getSize("default_margin").width - text: catalog.i18nc("@action:button", "Cancel") - onClicked: - { - overrideConfirmationDialog.reject() - } } ] From 01ecc799fad3567483ff835ece3c2efb42c48c29 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 13:06:28 +0100 Subject: [PATCH 11/24] Change background color of Discard or keepchanges dialog CURA-8951 --- resources/qml/Dialogs/DiscardOrKeepProfileChangesDialog.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/Dialogs/DiscardOrKeepProfileChangesDialog.qml b/resources/qml/Dialogs/DiscardOrKeepProfileChangesDialog.qml index 11aeac1722..0e2985ae32 100644 --- a/resources/qml/Dialogs/DiscardOrKeepProfileChangesDialog.qml +++ b/resources/qml/Dialogs/DiscardOrKeepProfileChangesDialog.qml @@ -20,7 +20,7 @@ UM.Dialog minimumHeight: UM.Theme.getSize("popup_dialog").height width: minimumWidth height: minimumHeight - + backgroundColor: UM.Theme.getColor("background_1") margin: UM.Theme.getSize("thick_margin").width property var changesModel: Cura.UserChangesModel { id: userChangesModel } From dab67d9b38fdef00cacb36a12f8769b180a89f38 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 13:26:16 +0100 Subject: [PATCH 12/24] Ensure that invisible menu seperators have a height of 0 CURA-9012 --- resources/qml/Widgets/MenuSeparator.qml | 1 + 1 file changed, 1 insertion(+) diff --git a/resources/qml/Widgets/MenuSeparator.qml b/resources/qml/Widgets/MenuSeparator.qml index ad79fccd6d..b39f958c69 100644 --- a/resources/qml/Widgets/MenuSeparator.qml +++ b/resources/qml/Widgets/MenuSeparator.qml @@ -18,4 +18,5 @@ MenuSeparator implicitHeight: UM.Theme.getSize("default_lining").height color: UM.Theme.getColor("setting_control_border") } + height: visible ? implicitHeight: 0 } \ No newline at end of file From d752f34b70ea519675be76d08fed04806587c36b Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 13:30:22 +0100 Subject: [PATCH 13/24] Shorten english string for show troubleshooting Doesn't solve all of the issues here, but at least fixes it for english. I feel that the word guide also doesn't actually add any meaningfull info anyway CURA-9012 --- resources/qml/Actions.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/Actions.qml b/resources/qml/Actions.qml index ec25eb8abb..90d1d549e2 100644 --- a/resources/qml/Actions.qml +++ b/resources/qml/Actions.qml @@ -79,7 +79,7 @@ Item { id: showTroubleShootingAction onTriggered: Qt.openUrlExternally("https://ultimaker.com/en/troubleshooting?utm_source=cura&utm_medium=software&utm_campaign=dropdown-troubleshooting") - text: catalog.i18nc("@action:inmenu", "Show Online Troubleshooting Guide") + text: catalog.i18nc("@action:inmenu", "Show Online Troubleshooting") } Action From 91b6ceb9512044f6ea4d3a66b03fe2cdf1bd0dec Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 14:01:54 +0100 Subject: [PATCH 14/24] Fix spacing for menu CURA-9012 --- resources/qml/MainWindow/ApplicationMenu.qml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/resources/qml/MainWindow/ApplicationMenu.qml b/resources/qml/MainWindow/ApplicationMenu.qml index bf3781331a..3598447d88 100644 --- a/resources/qml/MainWindow/ApplicationMenu.qml +++ b/resources/qml/MainWindow/ApplicationMenu.qml @@ -34,14 +34,16 @@ Item { text: menuBarItem.text.replace(new RegExp("&([A-Za-z])"), function (match, character) { - return `${character}`; + return `${character}` }) horizontalAlignment: Text.AlignLeft verticalAlignment: Text.AlignVCenter } - + leftPadding: UM.Theme.getSize("default_margin").width + rightPadding: UM.Theme.getSize("default_margin").width background: Rectangle { + color: menuBarItem.highlighted ? UM.Theme.getColor("background_2") : "transparent" } } From 6aeac2b2a4bb24042e5d8a32a462127992f849fc Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 16:12:27 +0100 Subject: [PATCH 15/24] Shorten copy of update button Added the full text in the tooltip CURA-9023 --- resources/qml/Preferences/ProfilesPage.qml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/resources/qml/Preferences/ProfilesPage.qml b/resources/qml/Preferences/ProfilesPage.qml index e5848a86c7..d1a05cba4c 100644 --- a/resources/qml/Preferences/ProfilesPage.qml +++ b/resources/qml/Preferences/ProfilesPage.qml @@ -325,9 +325,10 @@ UM.ManagementPage Cura.SecondaryButton { - text: catalog.i18nc("@action:button", "Update profile with current settings/overrides") + text: catalog.i18nc("@action:button", "Update profile.") enabled: Cura.MachineManager.hasUserSettings && objectList.currentIndex && !objectList.currentIndex.is_read_only onClicked: Cura.ContainerManager.updateQualityChanges() + tooltip: catalog.i18nc("@action:tooltip", "Update profile with current settings/overrides") } Cura.SecondaryButton From cbf995cb851c76aad2171350074b944787e799b2 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 16:24:49 +0100 Subject: [PATCH 16/24] Add tooltip to create new button CURA-9023 --- resources/qml/Preferences/ProfilesPage.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/Preferences/ProfilesPage.qml b/resources/qml/Preferences/ProfilesPage.qml index d1a05cba4c..b61aa3022c 100644 --- a/resources/qml/Preferences/ProfilesPage.qml +++ b/resources/qml/Preferences/ProfilesPage.qml @@ -76,7 +76,7 @@ UM.ManagementPage enabled: !Cura.MachineManager.stacksHaveErrors visible: base.canCreateProfile - + tooltip: catalog.i18nc("@action:tooltip", "Create new profile from current settings/overrides") onClicked: { createQualityDialog.object = Cura.ContainerManager.makeUniqueName(base.currentItem.name) From a285e39799ba90af0772263f29c33455bfb6913d Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 16:26:01 +0100 Subject: [PATCH 17/24] Set defaultname for create new profile CURA-9023 --- resources/qml/Preferences/ProfilesPage.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/Preferences/ProfilesPage.qml b/resources/qml/Preferences/ProfilesPage.qml index b61aa3022c..8982931b91 100644 --- a/resources/qml/Preferences/ProfilesPage.qml +++ b/resources/qml/Preferences/ProfilesPage.qml @@ -79,7 +79,7 @@ UM.ManagementPage tooltip: catalog.i18nc("@action:tooltip", "Create new profile from current settings/overrides") onClicked: { - createQualityDialog.object = Cura.ContainerManager.makeUniqueName(base.currentItem.name) + createQualityDialog.object = Cura.ContainerManager.makeUniqueName("") createQualityDialog.open() createQualityDialog.selectText() } From 84bb4790f808f726409b5dd1d2417450f89b659f Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 16:28:26 +0100 Subject: [PATCH 18/24] Fix spacing between update & discard actions CURA-9023 --- resources/qml/Preferences/ProfilesPage.qml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/resources/qml/Preferences/ProfilesPage.qml b/resources/qml/Preferences/ProfilesPage.qml index 8982931b91..a4069d4d82 100644 --- a/resources/qml/Preferences/ProfilesPage.qml +++ b/resources/qml/Preferences/ProfilesPage.qml @@ -316,13 +316,14 @@ UM.ManagementPage elide: Text.ElideRight } + Flow { id: currentSettingsActions width: parent.width visible: base.hasCurrentItem && base.currentItem.name == Cura.MachineManager.activeQualityOrQualityChangesName && base.currentItem.intent_category == Cura.MachineManager.activeIntentCategory - + spacing: UM.Theme.getSize("default_margin").width Cura.SecondaryButton { text: catalog.i18nc("@action:button", "Update profile.") From ec1f8f71250793d338489dc0fe7ef5158f34fcac Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Wed, 9 Mar 2022 16:30:32 +0100 Subject: [PATCH 19/24] Add extra info label CURA-9023 --- resources/qml/Preferences/ProfilesPage.qml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/resources/qml/Preferences/ProfilesPage.qml b/resources/qml/Preferences/ProfilesPage.qml index a4069d4d82..2a6b317314 100644 --- a/resources/qml/Preferences/ProfilesPage.qml +++ b/resources/qml/Preferences/ProfilesPage.qml @@ -315,7 +315,13 @@ UM.ManagementPage font: UM.Theme.getFont("large_bold") elide: Text.ElideRight } - + UM.Label + { + anchors.left: parent.left + anchors.right: parent.right + text: catalog.i18nc("@action:label", "Some settings from current profile were overwritten.") + visible: currentSettingsActions.visible + } Flow { From 84a0c732c1fa08f5dafcf4c27044da2d10b748f5 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Thu, 10 Mar 2022 11:34:11 +0100 Subject: [PATCH 20/24] Fix warnings caused by combobox Parent wasn't always set, which caused ocasional warnings --- resources/qml/Widgets/ComboBox.qml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/resources/qml/Widgets/ComboBox.qml b/resources/qml/Widgets/ComboBox.qml index 5d31348597..698eb4ec10 100644 --- a/resources/qml/Widgets/ComboBox.qml +++ b/resources/qml/Widgets/ComboBox.qml @@ -57,9 +57,7 @@ ComboBox contentItem: UM.Label { id: contentLabel - anchors.left: parent.left - anchors.leftMargin: UM.Theme.getSize("setting_unit_margin").width - anchors.verticalCenter: parent.verticalCenter + leftPadding: UM.Theme.getSize("setting_unit_margin").width + UM.Theme.getSize("default_margin").width anchors.right: downArrow.left wrapMode: Text.NoWrap text: From a5098f469b47244b0f59be48df4f20efb7e368b6 Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Thu, 10 Mar 2022 11:45:39 +0100 Subject: [PATCH 21/24] Lower the max flick velocity of settingview On my system it was 2500, which made it almost impossible to work with the trackpad CURA-9025 --- resources/qml/Settings/SettingView.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/qml/Settings/SettingView.qml b/resources/qml/Settings/SettingView.qml index 2ab98081e7..a2a5e0c36b 100644 --- a/resources/qml/Settings/SettingView.qml +++ b/resources/qml/Settings/SettingView.qml @@ -180,7 +180,7 @@ Item ListView { id: contents - + maximumFlickVelocity: 1000 anchors { top: filterContainer.bottom From dbcf57134015648a4ff9cae9e5b49189bfc831be Mon Sep 17 00:00:00 2001 From: Jaime van Kessel Date: Thu, 10 Mar 2022 11:54:58 +0100 Subject: [PATCH 22/24] Apply right theme colors for secondary button CURA-9014 --- resources/qml/SecondaryButton.qml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/resources/qml/SecondaryButton.qml b/resources/qml/SecondaryButton.qml index d8027af3fc..068d65c7b8 100644 --- a/resources/qml/SecondaryButton.qml +++ b/resources/qml/SecondaryButton.qml @@ -12,7 +12,8 @@ Cura.ActionButton color: UM.Theme.getColor("secondary_button") textColor: UM.Theme.getColor("secondary_button_text") outlineColor: UM.Theme.getColor("border_accent_1") - disabledColor: UM.Theme.getColor("action_button_disabled") + disabledColor: UM.Theme.getColor("secondary_button") textDisabledColor: UM.Theme.getColor("action_button_disabled_text") hoverColor: UM.Theme.getColor("secondary_button_hover") + outlineDisabledColor: UM.Theme.getColor("action_button_disabled_text") } \ No newline at end of file From d778b539c29c5890e04601d770c7b664b1a7af0e Mon Sep 17 00:00:00 2001 From: Ghostkeeper Date: Fri, 11 Mar 2022 10:13:32 +0100 Subject: [PATCH 23/24] Parse up to 16 extruders CuraEngine supports up to 16, so we should allow at least that. --- plugins/GCodeReader/FlavorParser.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/plugins/GCodeReader/FlavorParser.py b/plugins/GCodeReader/FlavorParser.py index ac24ce3184..555d9d594e 100644 --- a/plugins/GCodeReader/FlavorParser.py +++ b/plugins/GCodeReader/FlavorParser.py @@ -1,4 +1,4 @@ -# Copyright (c) 2021 Ultimaker B.V. +# Copyright (c) 2022 Ultimaker B.V. # Cura is released under the terms of the LGPLv3 or higher. import math @@ -31,6 +31,8 @@ Position = NamedTuple("Position", [("x", float), ("y", float), ("z", float), ("f class FlavorParser: """This parser is intended to interpret the common firmware codes among all the different flavors""" + MAX_EXTRUDER_COUNT = 16 + def __init__(self) -> None: CuraApplication.getInstance().hideMessageSignal.connect(self._onHideMessage) self._cancelled = False @@ -53,7 +55,7 @@ class FlavorParser: def _clearValues(self) -> None: self._extruder_number = 0 - self._extrusion_length_offset = [0] * 8 # type: List[float] + self._extrusion_length_offset = [0] * self.MAX_EXTRUDER_COUNT # type: List[float] self._layer_type = LayerPolygon.Inset0Type self._layer_number = 0 self._previous_z = 0 # type: float @@ -355,7 +357,7 @@ class FlavorParser: Logger.log("d", "Parsing g-code...") - current_position = Position(0, 0, 0, 0, [0] * 8) + current_position = Position(0, 0, 0, 0, [0] * self.MAX_EXTRUDER_COUNT) current_path = [] #type: List[List[float]] min_layer_number = 0 negative_layers = 0 From 25e914b1cbc3698844582913dd77d083f9739ca0 Mon Sep 17 00:00:00 2001 From: Ghostkeeper Date: Fri, 11 Mar 2022 10:25:31 +0100 Subject: [PATCH 24/24] Fix checkbox for automatic plug-in update checking The checkbox was still there in the preferences page, but it didn't work at all, for two reasons: - The preference entry didn't exist. This was causing warnings in the log (which is why I saw it). - It wasn't checking the preference before making that API call. Could make a bunch of people pretty angry. --- plugins/Marketplace/Marketplace.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/plugins/Marketplace/Marketplace.py b/plugins/Marketplace/Marketplace.py index 2d98947572..b9632e5fb0 100644 --- a/plugins/Marketplace/Marketplace.py +++ b/plugins/Marketplace/Marketplace.py @@ -31,8 +31,11 @@ class Marketplace(Extension, QObject): # Not entirely the cleanest code, since the localPackage list also checks the server if there are updates # Since that in turn will trigger notifications to be shown, we do need to construct it here and make sure # that it checks for updates... + preferences = CuraApplication.getInstance().getPreferences() + preferences.addPreference("info/automatic_plugin_update_check", True) self._local_package_list = LocalPackageList(self) - self._local_package_list.checkForUpdates(self._package_manager.local_packages) + if preferences.getValue("info/automatic_plugin_update_check"): + self._local_package_list.checkForUpdates(self._package_manager.local_packages) self._package_manager.installedPackagesChanged.connect(self.checkIfRestartNeeded)