Added basic fix for more options popup position

Librum-Reader · Jan 2, 2024 · abc013f · abc013f
1 parent 14f2cc4
commit abc013f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/presentation/homePage/MBook.qml b/src/presentation/homePage/MBook.qml
@@ -12,7 +12,7 @@ Item {
     property bool downloading: false
     signal leftButtonClicked(int index)
     signal rightButtonClicked(int index, var mouse)
-    signal moreOptionClicked(int index, var mouse)
+    signal moreOptionClicked(int index, var point)
 
     implicitWidth: 190
     implicitHeight: 322
@@ -222,8 +222,10 @@ Item {
     }
 
     MouseArea {
+        id: bookArea
         anchors.fill: parent
         acceptedButtons: Qt.AllButtons
+        cursorShape: Qt.PointingHandCursor
 
         // Delegate mouse clicks events to parent
         onClicked: mouse => {
@@ -283,10 +285,14 @@ Item {
                 id: moreOptionsArea
                 anchors.fill: parent
                 hoverEnabled: true
+                cursorShape: Qt.PointingHandCursor
 
                 onClicked: mouse => {
                                if (mouse.button === Qt.LeftButton) {
-                                   root.moreOptionClicked(root.index, mouse)
+                                   let point = mapToItem(bookArea,
+                                                         Qt.point(mouse.x,
+                                                                  mouse.y))
+                                   root.moreOptionClicked(root.index, point)
                                }
                            }
             }

diff --git a/src/presentation/homePage/MHomePage.qml b/src/presentation/homePage/MHomePage.qml
@@ -232,11 +232,10 @@ Page {
                         /*
                       When clicking more options, open the bookOptions popup
                       */
-                        onMoreOptionClicked: (index, mouse) => {
+                        onMoreOptionClicked: (index, point) => {
                                                  // Calculate where to spawn the bookOptions popup and set its position
                                                  let currentMousePosition = mapToItem(
-                                                     bookGridContainer,
-                                                     mouse.x, mouse.y)
+                                                     bookGridContainer, point)
 
                                                  bookOptionsPopup.x = currentMousePosition.x
                                                  - bookOptionsPopup.implicitWidth / 2