From 7f14c82d320303faee2593a2c414517d4f7634d0 Mon Sep 17 00:00:00 2001 From: Gabriel de Dietrich Date: Tue, 4 Mar 2014 12:56:49 +0100 Subject: MenuStyle: Fix popup margins for Desktop Change-Id: Iacf17af64cc9df0518a7b29a4a7817d7c69d7539 Reviewed-by: Jens Bache-Wiig --- src/controls/Styles/Desktop/MenuStyle.qml | 23 +++++++++-------------- 1 file changed, 9 insertions(+), 14 deletions(-) (limited to 'src/controls/Styles/Desktop/MenuStyle.qml') diff --git a/src/controls/Styles/Desktop/MenuStyle.qml b/src/controls/Styles/Desktop/MenuStyle.qml index b44dd70d..886fed3d 100644 --- a/src/controls/Styles/Desktop/MenuStyle.qml +++ b/src/controls/Styles/Desktop/MenuStyle.qml @@ -54,10 +54,6 @@ Style { property Component frame: StyleItem { elementType: "menu" - contentWidth: parent ? Math.round(parent.contentWidth) : 0 - contentHeight: parent ? Math.round(parent.contentHeight) : 0 - width: implicitWidth + 2 * (pixelMetric("menuhmargin") + pixelMetric("menupanelwidth")) - height: implicitHeight + 2 * (pixelMetric("menuvmargin") + pixelMetric("menupanelwidth")) Rectangle { visible: anchors.margins > 0 anchors { @@ -69,16 +65,15 @@ Style { Accessible.role: Accessible.PopupMenu - Binding { - target: styleRoot - property: "submenuOverlap" - value: 2 * pixelMetric("menupanelwidth") - } - - Binding { - target: styleRoot - property: "margin" - value: pixelMetric("menuvmargin") + pixelMetric("menupanelwidth") + Component.onCompleted: { + var menuHMargin = pixelMetric("menuhmargin") + var menuVMargin = pixelMetric("menuvmargin") + var menuPanelWidth = pixelMetric("menupanelwidth") + styleRoot.padding.left = menuHMargin + menuPanelWidth + styleRoot.padding.right = menuHMargin + menuPanelWidth + styleRoot.padding.top = menuVMargin + menuPanelWidth + styleRoot.padding.bottom = menuVMargin + menuPanelWidth + styleRoot.submenuOverlap = 2 * menuPanelWidth } // ### The Screen attached property can only be set on an Item, -- cgit v1.2.1