From 5c8ba7e29ee8620eaeaa7e039bb7d576704d8dc6 Mon Sep 17 00:00:00 2001 From: Joshua Goins Date: Sun, 25 May 2025 19:08:46 -0400 Subject: [PATCH] Fix space switching on mobile We forgot to connect to onTapped on the touchscreen-specific TapHandler. I also needed to carve out a specific case for DMs so it doesn't try to resolve the non-existent "DM" resource. --- src/app/qml/AvatarTabButton.qml | 1 + src/app/roommanager.cpp | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/app/qml/AvatarTabButton.qml b/src/app/qml/AvatarTabButton.qml index 12bfcc79a..31b76a9b1 100644 --- a/src/app/qml/AvatarTabButton.qml +++ b/src/app/qml/AvatarTabButton.qml @@ -48,6 +48,7 @@ Delegates.RoundedItemDelegate { TapHandler { acceptedDevices: PointerDevice.TouchScreen + onTapped: root.selected() onLongPressed: root.contextMenuRequested() } diff --git a/src/app/roommanager.cpp b/src/app/roommanager.cpp index 00c15a2f2..9964f1340 100644 --- a/src/app/roommanager.cpp +++ b/src/app/roommanager.cpp @@ -540,7 +540,9 @@ void RoomManager::setCurrentSpace(const QString &spaceId, bool setRoom) const auto &lastRoom = m_lastRoomConfig.readEntry(configSpaceId, QString()); if (lastRoom.isEmpty()) { - resolveResource(spaceId, "no_join"_L1); + if (spaceId != u"DM"_s) { + resolveResource(spaceId, "no_join"_L1); + } } else { resolveResource(lastRoom, "no_join"_L1); }