Don't allow long-pressing on non-touchscreen devices

It isn't the right kind of interaction on a computer with a mouse or trackpad, it should be relegated to touchscreen only. This should hopefully cover everything from room list delegates to messages.
2025-02-21 16:07:31 -05:00
parent 94fdf777cb
commit 88d684b6c1
9 changed files with 27 additions and 4 deletions
--- a/src/timeline/CodeComponent.qml
+++ b/src/timeline/CodeComponent.qml
@@ -128,7 +128,12 @@ QQC2.Control {
            TapHandler {
                enabled: root.time.toString() !== "Invalid Date"
                acceptedButtons: Qt.LeftButton
+                acceptedDevices: PointerDevice.Mouse | PointerDevice.TouchPad | PointerDevice.Stylus
                onTapped: RoomManager.maximizeCode(root.author, root.time, root.display, root.componentAttributes.class)
+            }
+
+            TapHandler {
+                acceptedDevices: PointerDevice.TouchScreen
                onLongPressed: root.showMessageMenu()
            }

--- a/src/timeline/HiddenDelegate.qml
+++ b/src/timeline/HiddenDelegate.qml
@@ -68,11 +68,13 @@ TimelineDelegate {

        TapHandler {
            acceptedButtons: Qt.RightButton
+            acceptedDevices: PointerDevice.Mouse | PointerDevice.TouchPad | PointerDevice.Stylus
            onTapped: _private.showMessageMenu()
        }

        TapHandler {
            acceptedButtons: Qt.LeftButton
+            acceptedDevices: PointerDevice.TouchScreen
            onLongPressed: _private.showMessageMenu()
        }

--- a/src/timeline/LiveLocationComponent.qml
+++ b/src/timeline/LiveLocationComponent.qml
@@ -69,6 +69,9 @@ ColumnLayout {
                });
                map.open();
            }
+        }
+        TapHandler {
+            acceptedDevices: PointerDevice.TouchScreen
            onLongPressed: openMessageContext("")
        }
        TapHandler {
--- a/src/timeline/LocationComponent.qml
+++ b/src/timeline/LocationComponent.qml
@@ -90,6 +90,9 @@ ColumnLayout {
                });
                map.open();
            }
+        }
+        TapHandler {
+            acceptedDevices: PointerDevice.TouchScreen
            onLongPressed: openMessageContext("")
        }
        TapHandler {
--- a/src/timeline/MessageDelegate.qml
+++ b/src/timeline/MessageDelegate.qml
@@ -323,6 +323,7 @@ TimelineDelegate {
            }

            TapHandler {
+                acceptedDevices: PointerDevice.TouchScreen
                acceptedButtons: Qt.LeftButton
                onLongPressed: _private.showMessageMenu()
            }
--- a/src/timeline/QuoteComponent.qml
+++ b/src/timeline/QuoteComponent.qml
@@ -55,6 +55,7 @@ QQC2.Control {

        TapHandler {
            enabled: !quoteText.hoveredLink
+            acceptedDevices: PointerDevice.TouchScreen
            acceptedButtons: Qt.LeftButton
            onLongPressed: root.showMessageMenu()
        }
--- a/src/timeline/TextComponent.qml
+++ b/src/timeline/TextComponent.qml
@@ -132,6 +132,7 @@ a{
    TapHandler {
        enabled: !root.hoveredLink
        acceptedButtons: Qt.LeftButton
+        acceptedDevices: PointerDevice.TouchScreen
        onLongPressed: root.showMessageMenu()
    }
 }