From 2d089f7d949d5e457025371b4921f8b35c8907e9 Mon Sep 17 00:00:00 2001
From: Tom Dewey <tom.dewey@raspberrypi.com>
Date: Tue, 3 Sep 2024 14:14:00 +0100
Subject: [PATCH] Popups: Don't close on click-outside

This was causing obvious confusion for users - misclicking outside of
the pop-up caused a dismissal where the consequences were not at all
clear.

So let's drop that behaviour entirely. You exit the dialog through
explicit button or key presses only.
---
 src/MsgPopup.qml | 2 +-
 src/main.qml     | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/MsgPopup.qml b/src/MsgPopup.qml
index 92252ae..d4002b7 100644
--- a/src/MsgPopup.qml
+++ b/src/MsgPopup.qml
@@ -16,7 +16,7 @@ Popup {
     width: 550
     height: msgpopupbody.implicitHeight+150
     padding: 0
-    closePolicy: Popup.CloseOnEscape | Popup.CloseOnPressOutside
+    closePolicy: Popup.CloseOnEscape
     modal: true
 
     property alias title: msgpopupheader.text
diff --git a/src/main.qml b/src/main.qml
index d644be1..a9fd6f6 100644
--- a/src/main.qml
+++ b/src/main.qml
@@ -437,7 +437,7 @@ ApplicationWindow {
         width: parent.width-100
         height: parent.height-50
         padding: 0
-        closePolicy: Popup.CloseOnEscape | Popup.CloseOnPressOutside
+        closePolicy: Popup.CloseOnEscape
         property string hwselected: ""
 
         // background of title
@@ -537,7 +537,7 @@ ApplicationWindow {
         width: parent.width-100
         height: parent.height-50
         padding: 0
-        closePolicy: Popup.CloseOnEscape | Popup.CloseOnPressOutside
+        closePolicy: Popup.CloseOnEscape
         property string categorySelected : ""
 
         // background of title
@@ -934,7 +934,7 @@ ApplicationWindow {
         width: parent.width-100
         height: parent.height-50
         padding: 0
-        closePolicy: Popup.CloseOnEscape | Popup.CloseOnPressOutside
+        closePolicy: Popup.CloseOnEscape
         onClosed: imageWriter.stopDriveListPolling()
 
         // background of title