ActionButton mouse area issue

Dirk Hohndel dirk at hohndel.org
Mon Feb 8 20:05:04 UTC 2016


On Mon, Feb 08, 2016 at 11:48:24AM -0800, Dirk Hohndel wrote:
> 
> I'm running into a pretty significant issues with the ActionButton:
> 
> Its MouseArea has an implicitWidth of parent.width. Which means that my page will not receive any clicks at all in the horizontal band that includes the ActionButton - so if the ActionButton "floats" above a ListView, the bottom most entry in the ListView cannot be selected (unless it happens to be taller than the icon size).
> 
> The issue appears to be that onClicked ALWAYS accepts the event. Even though the onClicked code simply returns if the click wasn't on the button (the math for that was just recently fixed) - the ListView "behind" never receives a click.

A patch like this might fix the problem (but I'm not sure if this would
break anything else.

/D

>From 4c2b08f9cd8d97a90994eac429515092bf539ee9 Mon Sep 17 00:00:00 2001
From: Dirk Hohndel <dirk at hohndel.org>
Date: Mon, 8 Feb 2016 12:01:25 -0800
Subject: [PATCH] The ActionButton MouseArea shouldn't be bigger than the button

Otherwise clicks to the bottom area of the screen don't get delivered to
whatever page the button "floats" in front of.

Signed-off-by: Dirk Hohndel <dirk at hohndel.org>
---
 components/mobilecomponents/qml/private/ActionButton.qml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/mobilecomponents/qml/private/ActionButton.qml b/components/mobilecomponents/qml/private/ActionButton.qml
index f65a3ed..6452228 100644
--- a/components/mobilecomponents/qml/private/ActionButton.qml
+++ b/components/mobilecomponents/qml/private/ActionButton.qml
@@ -30,7 +30,7 @@ MouseArea {
     //either Action or QAction should work here
     property QtObject action: pageStack.lastVisiblePage ? pageStack.lastVisiblePage.mainAction : null
 
-    implicitWidth: parent.width
+    implicitWidth: Units.iconSizes.medium
     implicitHeight: Units.iconSizes.medium
 
     drag {
-- 
1.9.1



More information about the Plasma-devel mailing list