Merge pull request #913 from Ultimaker/CURA-11102_qml_warnings

[CURA-11102] qml warnings
Ultimaker · Oct 10, 2023 · 5ad182b · 5ad182b
2 parents 75efad0 + 43707f5
commit 5ad182b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/UM/Qt/qml/UM/Menu.qml b/UM/Qt/qml/UM/Menu.qml
@@ -17,7 +17,7 @@ Menu
         {
             root.parent.visible = shouldBeVisible
             root.parent.height = shouldBeVisible ? UM.Theme.getSize("menu").height : 0
-            root.width = shouldBeVisible ? Qt.binding(setWidth) : 0
+            root.width = shouldBeVisible ? setWidth() : 0
         }
     }
 

diff --git a/UM/Qt/qml/UM/MenuItem.qml b/UM/Qt/qml/UM/MenuItem.qml
@@ -84,8 +84,7 @@ MenuItem
         {
             // Middle margin
             id: middleSpacer
-            width: visible ? UM.Theme.getSize("default_margin").width : 0
-            visible: _shortcut.nativeText != "" || root.subMenu
+            width: (_shortcut.nativeText != "" || root.subMenu) ? UM.Theme.getSize("default_margin").width : 0
         }
 
         UM.Label