From cd461da413514f8cac2700550cfbfed24e8a689f Mon Sep 17 00:00:00 2001 From: Tom Z Date: Thu, 19 Aug 2021 15:30:55 +0200 Subject: [PATCH] Fix hoverItem positioning. In the "Compact" layout the hover item is on top of the time label and the wrong mapping base caused a weird offset. Now we simply just position the hover item immediately above the current item. --- imports/NeoChat/Page/RoomPage.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imports/NeoChat/Page/RoomPage.qml b/imports/NeoChat/Page/RoomPage.qml index 9a4a777a7..5fab8a3fb 100644 --- a/imports/NeoChat/Page/RoomPage.qml +++ b/imports/NeoChat/Page/RoomPage.qml @@ -177,7 +177,7 @@ Kirigami.ScrollablePage { onTriggered: hoverActions.visible = hoverActions.visibleDelayed; } x: bubble ? (bubble.x + Kirigami.Units.largeSpacing + Math.max(bubble.width - childWidth, 0) - (Config.compactLayout ? Kirigami.Units.gridUnit * 3 : 0)) : 0 - y: bubble ? bubble.mapToItem(page, 0, -Kirigami.Units.largeSpacing - hoverActions.childHeight * 1.5).y : 0 + y: bubble ? bubble.mapToItem(parent, 0, 0).y - hoverActions.childHeight + Kirigami.Units.smallSpacing: 0; visible: false property var updateFunction