diff options
author | Kostas Karmas <konskarm@gmail.com> | 2020-08-03 18:30:21 +0300 |
---|---|---|
committer | Kostas Karmas <konskarm@gmail.com> | 2020-08-03 18:30:21 +0300 |
commit | 6c59616e39408c8af60a184120055f4a0257dbd6 (patch) | |
tree | e3a80ade5583eaa9e1ac92ba5b6dca207d4d68b5 /plugins/3MFReader/WorkspaceDialog.qml | |
parent | 60cfb6a8bea94fdb0f3a1d1df002282e0b4495cf (diff) |
Process feedback comments
Comments https://github.com/Ultimaker/Cura/pull/8142#discussion_r464309980 and https://github.com/Ultimaker/Cura/pull/8142#discussion_r464310281
CURA-7609
Diffstat (limited to 'plugins/3MFReader/WorkspaceDialog.qml')
-rw-r--r-- | plugins/3MFReader/WorkspaceDialog.qml | 35 |
1 files changed, 18 insertions, 17 deletions
diff --git a/plugins/3MFReader/WorkspaceDialog.qml b/plugins/3MFReader/WorkspaceDialog.qml index 48e93a3d3b..e4e02818b2 100644 --- a/plugins/3MFReader/WorkspaceDialog.qml +++ b/plugins/3MFReader/WorkspaceDialog.qml @@ -20,6 +20,7 @@ UM.Dialog property int comboboxHeight: 15 * screenScaleFactor property int spacerHeight: 10 * screenScaleFactor + property int doubleSpacerHeight: 20 * screenScaleFactor property string machineResolveStrategyCurrentKey: "override" onClosing: manager.notifyClosed() @@ -36,7 +37,7 @@ UM.Dialog Item { anchors.fill: parent - anchors.margins: 20 * screenScaleFactor + anchors.margins: 10 * screenScaleFactor UM.I18nCatalog { @@ -80,7 +81,7 @@ UM.Dialog } Item // Spacer { - height: 2 * spacerHeight + height: doubleSpacerHeight width: height } @@ -105,7 +106,7 @@ UM.Dialog id: machineResolveStrategyTooltip width: (parent.width / 3) | 0 height: visible ? comboboxHeight : 0 - visible: manager.updatableMachinesCount != 0 + visible: manager.updatableMachineNamesCount != 0 text: catalog.i18nc("@info:tooltip", "How should the conflict in the machine be resolved?") ComboBox { @@ -163,12 +164,12 @@ UM.Dialog id: machineResolveTooltip width: (parent.width / 3) | 0 height: visible ? comboboxHeight : 0 - visible: base.visible && manager.updatableMachinesCount != 0 && machineResolveStrategyCurrentKey == "override" + visible: base.visible && manager.updatableMachineNamesCount != 0 && machineResolveStrategyCurrentKey == "override" text: catalog.i18nc("@info:tooltip", "Which machine of the same type should be overriden?") ComboBox { id: selectMachineComboBox - model: manager.updatableMachines + model: manager.updatableMachineNames width: parent.width onCurrentIndexChanged: { @@ -176,18 +177,18 @@ UM.Dialog } onVisibleChanged: { - if (visible) + if (!visible) {return} + + currentIndex = 0 + for (var i = 0; i < count; i++) { - currentIndex = 0 - for (var i = 0; i < count; i++) + if (model[i] == manager.machineName) { - if (model[i] == manager.machineName) - { - currentIndex = i - break - } + currentIndex = i + break } } + } } } @@ -195,7 +196,7 @@ UM.Dialog Item // Spacer { - height: 2 * spacerHeight + height: doubleSpacerHeight width: height } Row @@ -298,7 +299,7 @@ UM.Dialog } Item // Spacer { - height: 2 * spacerHeight + height: doubleSpacerHeight width: height } Row @@ -360,7 +361,7 @@ UM.Dialog Item // Spacer { - height: 2 * spacerHeight + height: doubleSpacerHeight width: height } @@ -402,7 +403,7 @@ UM.Dialog } Item // Spacer { - height: 2 * spacerHeight + height: spacerHeight width: height } Row |