From 6f3ebafc4629a51445380973754f8bbaad1d3061 Mon Sep 17 00:00:00 2001
From: BlueGreenMagick <50060875+BlueGreenMagick@users.noreply.github.com>
Date: Tue, 26 May 2020 18:27:38 +0900
Subject: [PATCH] fix drag drop field repositioning

---
 qt/aqt/fields.py | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/qt/aqt/fields.py b/qt/aqt/fields.py
index 55b854063..f0c6fb03b 100644
--- a/qt/aqt/fields.py
+++ b/qt/aqt/fields.py
@@ -59,12 +59,17 @@ class FieldDialog(QDialog):
         fieldList = self.form.fieldList
         indicatorPos = fieldList.dropIndicatorPosition()
         dropPos = fieldList.indexAt(ev.pos()).row()
-        if indicatorPos == QAbstractItemView.OnViewport:  # to bottom
-            self.moveField(fieldList.count())
+        idx = self.currentIdx
+        if indicatorPos == QAbstractItemView.OnViewport:  # to bottom.
+            movePos = fieldList.count() - 1
         elif indicatorPos == QAbstractItemView.AboveItem:
-            self.moveField(dropPos)
+            movePos = dropPos
         elif indicatorPos == QAbstractItemView.BelowItem:
-            self.moveField(dropPos + 1)
+            movePos = dropPos + 1
+        # the item in idx is removed thus subtract 1.
+        if idx < dropPos:
+            movePos -= 1
+        self.moveField(movePos + 1)  # convert to 1 based.
 
     def onRowChange(self, idx):
         if idx == -1: