Bug 260893

Summary:	"long press" action menus aren't hidden
Product:	[Unmaintained] KMail Mobile	Reporter:	Bjoern Ricks <bjoern.ricks>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	VERIFIED FIXED
Severity:	normal	CC:	aheinecke, kde
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Maemo 5
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Bjoern Ricks 2010-12-21 16:54:47 UTC

Menus that are opened via a long press action e.g. the different reply actions aren't closed of the user select e.g. the list view.

Steps to reproduce:
1) Open mail
2) select a mail in the list view
3) open actions menu
4) long press reply
5) press "back to list view" button

Result:
All reply buttons are still shown. It's the same for all "long press" button menus (e.g. Mark Email as too).

Behavior is the same on wince and maemo.

Comment 1 Tobias Koenig 2011-01-03 15:41:12 UTC

commit 286017e64a62569f3aa9e8d05b5ce5f5382cca4e
branch master
Author: Tobias Koenig <tokoe@kde.org>
Date:   Mon Jan 3 15:10:26 2011 +0100

    Use opaque background for action pages
    
    Use an opaque background for action pages and
    intercept mouse events to prevent pressing the
    'back to list' button accidentally.
    
    BUG: 260893

diff --git a/mobile/mail/ForwardOptionsPage.qml b/mobile/mail/ForwardOptionsPage.qml
index 5cbba1c..fcfb910 100644
--- a/mobile/mail/ForwardOptionsPage.qml
+++ b/mobile/mail/ForwardOptionsPage.qml
@@ -24,9 +24,16 @@ import Qt 4.7 as QML
 import org.kde 4.5
 import org.kde.pim.mobileui 4.5 as KPIM
 
-QML.Item {
+QML.Rectangle {
   id: root
   anchors.fill: parent
+  color: "white"
+  z: 10
+
+  QML.MouseArea {
+    anchors.fill : parent
+    onClicked : {} // do nothing
+  }
 
   QML.Rectangle {
     anchors.right : parent.right
diff --git a/mobile/mail/MarkAsPage.qml b/mobile/mail/MarkAsPage.qml
index 2ee2afd..7830b08 100644
--- a/mobile/mail/MarkAsPage.qml
+++ b/mobile/mail/MarkAsPage.qml
@@ -24,9 +24,16 @@ import Qt 4.7 as QML
 import org.kde 4.5
 import org.kde.pim.mobileui 4.5 as KPIM
 
-QML.Item {
+QML.Rectangle {
   id: root
   anchors.fill: parent
+  color: "white"
+  z: 10
+
+  QML.MouseArea {
+    anchors.fill : parent
+    onClicked : {} // do nothing
+  }
 
   QML.Rectangle {
     anchors.right : root.right
diff --git a/mobile/mail/NewMailPage.qml b/mobile/mail/NewMailPage.qml
index 9dc6c98..54b38de 100644
--- a/mobile/mail/NewMailPage.qml
+++ b/mobile/mail/NewMailPage.qml
@@ -24,9 +24,17 @@ import Qt 4.7 as QML
 import org.kde 4.5
 import org.kde.pim.mobileui 4.5 as KPIM
 
-QML.Item { // work around strange anchoring behaviour inside Loader
+QML.Rectangle { // work around strange anchoring behaviour inside Loader
   id : root
   anchors.fill: parent
+  color: "white"
+  z: 10
+
+  QML.MouseArea {
+    anchors.fill : parent
+    onClicked : {} // do nothing
+  }
+
   QML.Rectangle {
     anchors.right : parent.right
     anchors.rightMargin : 70
diff --git a/mobile/mail/ReplyOptionsPage.qml b/mobile/mail/ReplyOptionsPage.qml
index 9456d40..57dd217 100644
--- a/mobile/mail/ReplyOptionsPage.qml
+++ b/mobile/mail/ReplyOptionsPage.qml
@@ -24,9 +24,16 @@ import Qt 4.7 as QML
 import org.kde 4.5
 import org.kde.pim.mobileui 4.5 as KPIM
 
-QML.Item {
+QML.Rectangle {
   id: root
   anchors.fill: parent
+  color: "white"
+  z: 10
+
+  QML.MouseArea {
+    anchors.fill : parent
+    onClicked : {} // do nothing
+  }
 
   QML.Rectangle {
     anchors.right : parent.right