diff --git a/resources/qml/Menus/MaterialBrandMenu.qml b/resources/qml/Menus/MaterialBrandMenu.qml index 5a5838f4ca..deb1021cf5 100644 --- a/resources/qml/Menus/MaterialBrandMenu.qml +++ b/resources/qml/Menus/MaterialBrandMenu.qml @@ -14,6 +14,7 @@ Instead we'll use a pop-up which doesn't seem to have that problem. */ Cura.MenuItem { id: materialBrandMenu + height: UM.Theme.getSize("menu").height + UM.Theme.getSize("narrow_margin").height overrideShowArrow: true property var materialTypesModel diff --git a/resources/qml/Menus/MaterialBrandSubMenu.qml b/resources/qml/Menus/MaterialBrandSubMenu.qml index bc02e79cdb..318afd9c60 100644 --- a/resources/qml/Menus/MaterialBrandSubMenu.qml +++ b/resources/qml/Menus/MaterialBrandSubMenu.qml @@ -12,11 +12,13 @@ Popup { id: materialBrandSubMenu - bottomPadding: UM.Theme.getSize("thin_margin").height + // There is a bug where hovering the bottom half of the last element causes the popup to close. + // Undo this commit if you find a fix. + bottomPadding: -UM.Theme.getSize("thin_margin").height topPadding: UM.Theme.getSize("thin_margin").height implicitWidth: scrollViewContent.width + scrollbar.width + leftPadding + rightPadding - implicitHeight: scrollViewContent.height + bottomPadding + topPadding + implicitHeight: scrollViewContent.height + bottomPadding + topPadding + (2 * UM.Theme.getSize("thin_margin").height) // offset position relative to the parent property int implicitX: parent.width - UM.Theme.getSize("default_lining").width