make deleting history item a remorse action

rabauke · Jan 27, 2024 · 3710f6c · 3710f6c
1 parent 06bffb4
commit 3710f6c
Showing 1 changed file with 37 additions and 36 deletions.
diff --git a/qml/pages/History.qml b/qml/pages/History.qml
@@ -39,10 +39,46 @@ Page {
     model: searchHistoryListModel
 
     delegate: ListItem {
+      id: listItem
       width: parent.width
       contentWidth: parent.width
       contentHeight: query_text.height + Theme.paddingLarge
-      menu: contextMenu
+
+      menu: ContextMenu {
+        MenuItem {
+          text: qsTr('Search again')
+          onClicked: {
+            queryFieldText = searchHistoryListModel.get(model.index).query
+            resultsListModel.clear()
+            var transAtoB = appModel.dictionary.translateAtoB(queryFieldText)
+            var transBtoA = appModel.dictionary.translateBtoA(queryFieldText)
+            var i
+            for (i in transAtoB)
+              resultsListModel.append({
+                                        'section': 'AtoB',
+                                        'lang1': transAtoB[i][0],
+                                        'lang2': transAtoB[i][1]
+                                      })
+            for (i in transBtoA)
+              resultsListModel.append({
+                                        'section': 'BtoA',
+                                        'lang1': transBtoA[i][0],
+                                        'lang2': transBtoA[i][1]
+                                      })
+            numberOfResultsAtoB = transAtoB.length
+            numberOfResultsBtoA = transBtoA.length
+            queryFieldText = searchHistoryListModel.get(model.index).query
+            pageStack.navigateBack()
+          }
+        }
+        MenuItem {
+          text: qsTr('Remove query')
+          onClicked: listItem.remorseDelete(function () {
+            searchHistoryListModel.remove(model.index)
+          })
+        }
+      }
+
       Text {
         id: query_text
         x: Theme.horizontalPageMargin
@@ -54,41 +90,6 @@ Page {
         horizontalAlignment: TextEdit.AlignLeft
         text: query
       }
-      Component {
-        id: contextMenu
-        ContextMenu {
-          MenuItem {
-            text: qsTr('Search again')
-            onClicked: {
-              queryFieldText = searchHistoryListModel.get(model.index).query
-              resultsListModel.clear()
-              var transAtoB = appModel.dictionary.translateAtoB(queryFieldText)
-              var transBtoA = appModel.dictionary.translateBtoA(queryFieldText)
-              var i
-              for (i in transAtoB)
-                resultsListModel.append({
-                                          'section': 'AtoB',
-                                          'lang1': transAtoB[i][0],
-                                          'lang2': transAtoB[i][1]
-                                        })
-              for (i in transBtoA)
-                resultsListModel.append({
-                                          'section': 'BtoA',
-                                          'lang1': transBtoA[i][0],
-                                          'lang2': transBtoA[i][1]
-                                        })
-              numberOfResultsAtoB = transAtoB.length
-              numberOfResultsBtoA = transBtoA.length
-              queryFieldText = searchHistoryListModel.get(model.index).query
-              pageStack.navigateBack()
-            }
-          }
-          MenuItem {
-            text: qsTr('Remove query')
-            onClicked: searchHistoryListModel.remove(model.index)
-          }
-        }
-      }
     }
   }
 }