pygpt-net 2.6.59__py3-none-any.whl → 2.6.61__py3-none-any.whl

pygpt_net/ui/widget/lists/preset.py

@@ -6,12 +6,12 @@
 # GitHub:  https://github.com/szczyglis-dev/py-gpt   #
 # MIT License                                        #
 # Created By  : Marcin Szczygliński                  #
-# Updated Date: 2025.08.24 23:00:00                  #
+# Updated Date: 2025.09.26 03:00:00                  #
 # ================================================== #
 
-from PySide6.QtCore import QPoint, QItemSelectionModel
-from PySide6.QtGui import QAction, QIcon, Qt
-from PySide6.QtWidgets import QMenu
+from PySide6.QtCore import QPoint, QItemSelectionModel, Qt, QEventLoop, QTimer, QMimeData
+from PySide6.QtGui import QAction, QIcon, QCursor, QDrag, QPainter, QPixmap, QPen, QColor
+from PySide6.QtWidgets import QMenu, QAbstractItemView, QApplication
 
 from pygpt_net.core.types import (
     MODE_EXPERT,
@@ -27,6 +27,12 @@ class PresetList(BaseList):
     _ICO_DELETE = QIcon(":/icons/delete.svg")
     _ICO_CHECK = QIcon(":/icons/check.svg")
     _ICO_CLOSE = QIcon(":/icons/close.svg")
+    _ICO_UP = QIcon(":/icons/collapse.svg")
+    _ICO_DOWN = QIcon(":/icons/expand.svg")
+
+    ROLE_UUID = Qt.UserRole + 1
+    ROLE_ID = Qt.UserRole + 2
+    ROLE_IS_SPECIAL = Qt.UserRole + 3
 
     def __init__(self, window=None, id=None):
         """
@@ -45,40 +51,175 @@ class PresetList(BaseList):
         self._backup_selection = None
         self.restore_after_ctx_menu = True
 
+        # Flat list behavior
+        self.setRootIsDecorated(False)
+        self.setItemsExpandable(False)
+        self.setUniformRowHeights(True)
+        self.setSelectionBehavior(QAbstractItemView.SelectRows)
+        self.setSelectionMode(QAbstractItemView.SingleSelection)
+
+        # Drag & drop state
+        self._dnd_enabled = False
+        self.setDragEnabled(False)
+        self.setAcceptDrops(False)
+        self.setDragDropMode(QAbstractItemView.NoDragDrop)  # switched dynamically
+        self.setDefaultDropAction(Qt.MoveAction)
+        self.setDragDropOverwriteMode(False)
+        self.setDropIndicatorShown(False)
+
+        self._press_pos = None
+        self._press_index = None
+        self._press_backup_selection = None
+        self._press_backup_current = None
+        self._dragging = False
+        self._dragged_was_selected = False
+
+        # Mark that we already applied selection at drag start (one-shot per DnD)
+        self._drag_selection_applied = False
+
+        # ID-based selection persistence (single selection list)
+        self._saved_selection_ids = None
+
+        # Defer refresh payload after drop (DnD teardown must finish first)
+        self._pending_after_drop = None
+
+        # Guard against input during model rebuild (prevents crashes on quick clicks)
+        self._model_updating = False
+
+        # Keep original selection IDs before opening context menu (right-click)
+        self._ctx_menu_original_ids = None
+
+        # One-shot forced selection after refresh (list of ROLE_ID)
+        self._selection_override_ids = None
+
+    # -------- Public helpers to protect updates --------
+
+    def begin_model_update(self):
+        """Temporarily block user interaction while the model/view is rebuilt."""
+        self._model_updating = True
+        self.setEnabled(False)
+
+    def end_model_update(self):
+        """Re-enable interaction after model/view rebuild is complete."""
+        self.setEnabled(True)
+        self._model_updating = False
+
+    # ---------------------------------------------------
+
+    def set_dnd_enabled(self, enabled: bool):
+        """
+        Toggle DnD behaviour at runtime.
+        Using DragDrop (not InternalMove) to avoid implicit Qt reordering.
+        """
+        self._dnd_enabled = bool(enabled)
+        if self._dnd_enabled:
+            self.setDragEnabled(True)
+            self.setAcceptDrops(True)
+            self.setDragDropMode(QAbstractItemView.DragDrop)
+            self.setDropIndicatorShown(True)
+        else:
+            self.setDragEnabled(False)
+            self.setAcceptDrops(False)
+            self.setDragDropMode(QAbstractItemView.NoDragDrop)
+            self.setDropIndicatorShown(False)
+            self.unsetCursor()
+
+    def backup_selection(self):
+        """
+        Persist selected preset identity (by ROLE_ID) instead of raw indexes.
+        """
+        try:
+            sel_rows = self.selectionModel().selectedRows()
+            ids = []
+            for ix in sel_rows:
+                pid = ix.data(self.ROLE_ID)
+                if pid:
+                    ids.append(str(pid))
+            self._saved_selection_ids = ids if ids else None
+        except Exception:
+            self._saved_selection_ids = None
+
+    def restore_selection(self):
+        """
+        Restore selection by ROLE_ID to keep it attached to the same item regardless of position.
+        """
+        ids = self._saved_selection_ids or []
+        self._saved_selection_ids = None
+        if not ids:
+            return
+        model = self.model()
+        if model is None:
+            return
+        target_id = ids[0]
+        sel_model = self.selectionModel()
+        prev_unlocked = self.unlocked
+        self.unlocked = True
+        try:
+            sel_model.clearSelection()
+            first_idx = None
+            for r in range(model.rowCount()):
+                idx = model.index(r, 0)
+                pid = idx.data(self.ROLE_ID)
+                if pid == target_id:
+                    first_idx = idx
+                    break
+            if first_idx is not None and first_idx.isValid():
+                sel_model.select(first_idx, QItemSelectionModel.ClearAndSelect | QItemSelectionModel.Rows)
+                self.setCurrentIndex(first_idx)
+                self.scrollTo(first_idx)
+        finally:
+            self.unlocked = prev_unlocked
+
+    def _current_selected_ids(self) -> list[str]:
+        """Read current selection IDs (ROLE_ID)."""
+        try:
+            return [ix.data(self.ROLE_ID) for ix in self.selectionModel().selectedRows() if ix.data(self.ROLE_ID)]
+        except Exception:
+            return []
+
     def click(self, val):
-        row = val.row()
+        """Row click handler; select by ID (stable under reordering)."""
+        if self._model_updating:
+            return
+        index = val
+        if not index.isValid():
+            return
+        preset_id = index.data(self.ROLE_ID)
+        if preset_id:
+            self.window.controller.presets.select_by_id(preset_id)
+            self.selection = self.selectionModel().selection()
+            return
+        row = index.row()
         if row >= 0:
             self.window.controller.presets.select(row)
             self.selection = self.selectionModel().selection()
 
     def dblclick(self, val):
-        """
-        Double click event
-
-        :param val: double click event
-        """
+        """Double click event"""
+        if self._model_updating:
+            return
         row = val.row()
         if row >= 0:
             self.window.controller.presets.editor.edit(row)
 
     def show_context_menu(self, pos: QPoint):
-        """
-        Context menu event
-
-        :param pos: QPoint
-        """
+        """Context menu event"""
+        if self._model_updating:
+            return
         global_pos = self.viewport().mapToGlobal(pos)
         mode = self.window.core.config.get('mode')
         index = self.indexAt(pos)
         idx = index.row()
 
         preset = None
+        preset_id = None
         if idx >= 0:
             preset_id = self.window.core.presets.get_by_idx(idx, mode)
             if preset_id:
                 preset = self.window.core.presets.items.get(preset_id)
 
         is_current = idx >= 0 and self.window.controller.presets.is_current(idx)
+        is_special = bool(index.data(self.ROLE_IS_SPECIAL)) if index.isValid() else False
 
         if idx >= 0:
             menu = QMenu(self)
@@ -100,6 +241,16 @@ class PresetList(BaseList):
             duplicate_act = QAction(self._ICO_COPY, trans('preset.action.duplicate'), menu)
             duplicate_act.triggered.connect(lambda checked=False, it=index: self.action_duplicate(it))
 
+            if self._dnd_enabled and not is_current and not is_special:
+                up_act = QAction(self._ICO_UP, trans('common.up'), menu)
+                down_act = QAction(self._ICO_DOWN, trans('common.down'), menu)
+                up_act.setEnabled(idx > 1)
+                down_act.setEnabled(idx < (self.model().rowCount() - 1))
+                up_act.triggered.connect(lambda checked=False, it=index: self.action_move_up(it))
+                down_act.triggered.connect(lambda checked=False, it=index: self.action_move_down(it))
+                menu.addAction(up_act)
+                menu.addAction(down_act)
+
             if is_current:
                 edit_act.setEnabled(False)
                 restore_act = QAction(self._ICO_UNDO, trans('dialog.editor.btn.defaults'), menu)
@@ -115,10 +266,7 @@ class PresetList(BaseList):
             self.selection = self.selectionModel().selection()
             menu.exec_(global_pos)
 
-        # store previous scroll position
         self.store_scroll_position()
-
-        # restore selection if it was backed up
         if self.restore_after_ctx_menu:
             if self._backup_selection is not None:
                 sel_model = self.selectionModel()
@@ -128,82 +276,340 @@ class PresetList(BaseList):
                         i, QItemSelectionModel.Select | QItemSelectionModel.Rows
                     )
                 self._backup_selection = None
-
-        # restore scroll position
         self.restore_after_ctx_menu = True
         self.restore_scroll_position()
 
     def action_edit(self, item):
-        """
-        Edit action handler
-
-        :param item: list item
-        """
         idx = item.row()
         if idx >= 0:
-            self.restore_after_ctx_menu = False  # do not restore selection after context menu
+            self.restore_after_ctx_menu = False
             self.window.controller.presets.editor.edit(idx)
 
     def action_duplicate(self, item):
-        """
-        Duplicate action handler
-
-        :param item: list item
-        """
         idx = item.row()
         if idx >= 0:
-            self.restore_after_ctx_menu = False  # do not restore selection after context menu
+            self.restore_after_ctx_menu = False
             self.window.controller.presets.duplicate(idx)
 
     def action_delete(self, item):
-        """
-        Delete action handler
-
-       :param item: list item
-        """
         idx = item.row()
         if idx >= 0:
-            self.restore_after_ctx_menu = False  # do not restore selection after context menu
+            self.restore_after_ctx_menu = False
             self.window.controller.presets.delete(idx)
 
     def action_restore(self, item):
-        """
-        Restore action handler
-
-        :param item: list item
-        """
         self.window.controller.presets.restore()
 
     def action_enable(self, item):
-        """
-        Enable action handler
-
-        :param item: list item
-        """
         idx = item.row()
         if idx >= 0:
             self.window.controller.presets.enable(idx)
 
     def action_disable(self, item):
-        """
-        Disable action handler
-
-        :param item: list item
-        """
         idx = item.row()
         if idx >= 0:
             self.window.controller.presets.disable(idx)
 
+    def action_move_up(self, item):
+        row = item.row()
+        if row <= 1:
+            return
+        self.restore_after_ctx_menu = False
+        # Select the moved element (exception rule for context Up)
+        moved_role_id = item.data(self.ROLE_ID)
+        if moved_role_id:
+            self._selection_override_ids = [moved_role_id]
+            # Keep controller in sync with the view selection
+            self.window.controller.presets.select_by_id(moved_role_id)
+        self._move_row(row, row - 1)
+
+    def action_move_down(self, item):
+        row = item.row()
+        if row < 0 or row >= (self.model().rowCount() - 1):
+            return
+        if row == 0:
+            return
+        self.restore_after_ctx_menu = False
+        # Select the moved element (exception rule for context Down)
+        moved_role_id = item.data(self.ROLE_ID)
+        if moved_role_id:
+            self._selection_override_ids = [moved_role_id]
+            # Keep controller in sync with the view selection
+            self.window.controller.presets.select_by_id(moved_role_id)
+        self._move_row(row, row + 1)
+
+    # ----------------------------
+    # Ordering helpers (core-based)
+    # ----------------------------
+
+    def _core_regular_ids_for_mode(self) -> list[str]:
+        """Return current ordered preset IDs for mode, excluding pinned current.<mode>."""
+        mode = self.window.core.config.get('mode')
+        data = self.window.core.presets.get_by_mode(mode) or {}
+        ids = list(data.keys())
+        if ids and ids[0].startswith("current."):
+            ids = ids[1:]
+        return ids
+
+    def _core_regular_uuids_for_mode(self) -> list[str]:
+        """UUID list resolved from core ordered IDs (excluding pinned)."""
+        ids = self._core_regular_ids_for_mode()
+        items = self.window.core.presets.items
+        out = []
+        for pid in ids:
+            it = items.get(pid)
+            if it and it.uuid:
+                out.append(it.uuid)
+        return out
+
+    def _collect_regular_uuids(self) -> list[str]:
+        """Backward-compatible wrapper used by older code: now returns core-based UUIDs."""
+        return self._core_regular_uuids_for_mode()
+
+    def _is_row_selected(self, row: int) -> bool:
+        """Check if given row is currently selected."""
+        try:
+            sel = self.selectionModel().selectedRows()
+            return any(ix.row() == row for ix in sel)
+        except Exception:
+            return False
+
+    def _reorder_and_persist(self, from_row: int, to_row: int) -> str:
+        """
+        Compute new UUID order using core order (not the view), then persist it.
+        Returns moved preset ID (filename) for later selection if needed.
+        """
+        if from_row <= 0 or to_row <= 0:
+            return ""
+
+        ids_seq = self._core_regular_ids_for_mode()
+        if not ids_seq:
+            return ""
+
+        i_from = from_row - 1
+        i_to = to_row - 1
+        if i_from < 0 or i_from >= len(ids_seq):
+            return ""
+        if i_to < 0:
+            i_to = 0
+        if i_to > len(ids_seq):
+            i_to = len(ids_seq)
+
+        moved_id = ids_seq[i_from]
+        seq_ids = list(ids_seq)
+        item = seq_ids.pop(i_from)
+        seq_ids.insert(i_to if i_to <= len(seq_ids) else len(seq_ids), item)
+
+        items = self.window.core.presets.items
+        uuids = [items[pid].uuid for pid in seq_ids if pid in items and items[pid].uuid]
+        mode = self.window.core.config.get('mode')
+        self.window.controller.presets.persist_order_for_mode(mode, uuids)
+
+        return moved_id
+
+    # ----------------------------
+    # Drag visuals (safe, no delegate painting)
+    # ----------------------------
+
+    def _drag_pixmap_for_index(self, index) -> QPixmap | None:
+        """
+        Build a safe pixmap for dragged row without using delegate.paint (prevents crash).
+        """
+        try:
+            text = str(index.data(Qt.DisplayRole) or "")
+            fm = self.fontMetrics()
+            w = max(fm.horizontalAdvance(text) + 24, 80)
+            h = max(fm.height() + 10, 24)
+            pm = QPixmap(w, h)
+            pm.fill(Qt.transparent)
+
+            painter = QPainter()
+            painter.begin(pm)
+            try:
+                # background bubble
+                bg = self.palette().base().color()
+                bg.setAlpha(220)
+                painter.fillRect(pm.rect(), bg)
+                # border
+                pen = QPen(QColor(0, 0, 0, 40))
+                painter.setPen(pen)
+                painter.drawRect(pm.rect().adjusted(0, 0, -1, -1))
+                # text
+                painter.setPen(self.palette().text().color())
+                painter.drawText(pm.rect().adjusted(8, 0, -8, 0), Qt.AlignVCenter | Qt.AlignLeft, text)
+            finally:
+                painter.end()
+            return pm
+        except Exception:
+            return None
+
+    def startDrag(self, supportedActions):
+        """
+        Start drag with pixmap built from the actually dragged row (self._press_index).
+        Avoids using selection for drag visuals (no 'ghost' of another item).
+        """
+        if not self._dnd_enabled or self._press_index is None or not self._press_index.isValid():
+            return super().startDrag(supportedActions)
+
+        model = self.model()
+        drag = QDrag(self)
+        # mime data from the pressed index (not from selection)
+        try:
+            mime = model.mimeData([self._press_index])
+        except Exception:
+            mime = QMimeData()
+        drag.setMimeData(mime)
+
+        pm = self._drag_pixmap_for_index(self._press_index)
+        if pm is not None:
+            drag.setPixmap(pm)
+            drag.setHotSpot(pm.rect().center())
+
+        drag.exec(Qt.MoveAction)
+
+    # ----------------------------
+    # Refresh & painting
+    # ----------------------------
+
+    def _force_full_repaint(self):
+        """
+        Force a synchronous full repaint of the viewport and notify the view that data/layout could change.
+        This clears any stale drag visuals on some platforms/styles.
+        """
+        model = self.model()
+        if model is not None and model.rowCount() > 0:
+            top = model.index(0, 0)
+            bottom = model.index(model.rowCount() - 1, 0)
+            try:
+                model.dataChanged.emit(top, bottom, [Qt.DisplayRole])
+            except Exception:
+                pass
+            try:
+                model.layoutChanged.emit()
+            except Exception:
+                pass
+        self.viewport().repaint()
+
+    def _refresh_after_order_change(self, moved_id: str, follow_selection: bool):
+        """
+        Refresh the list from core order and keep selection/scroll stable.
+
+        For both DnD and context moves:
+        - if _selection_override_ids is set, layout will restore those IDs;
+        - otherwise, take current selected IDs and use them as override to ensure
+          selection 'follows element, not position'.
+        """
+        if not self._selection_override_ids:
+            self._selection_override_ids = self._current_selected_ids()
+
+        self.store_scroll_position()
+
+        di_prev = self._dnd_enabled
+        self.setDropIndicatorShown(False)
+        self.setUpdatesEnabled(False)
+        try:
+            self.window.controller.presets.update_list()
+            self.restore_scroll_position()
+        finally:
+            self.setUpdatesEnabled(True)
+            if di_prev and self._dnd_enabled:
+                self.setDropIndicatorShown(True)
+
+        # Clear helpers for context menu (layout will consume _selection_override_ids)
+        self._ctx_menu_original_ids = None
+        self._backup_selection = None
+
+        QApplication.processEvents(QEventLoop.ExcludeUserInputEvents | QEventLoop.ExcludeSocketNotifiers)
+        self._force_full_repaint()
+        QTimer.singleShot(0, self.viewport().update)
+
+    def _apply_after_drop(self):
+        """Execute deferred refresh after the drop event has fully finished in Qt."""
+        payload = self._pending_after_drop
+        self._pending_after_drop = None
+        if not payload:
+            return
+        moved_id, follow_selection = payload
+        self._refresh_after_order_change(moved_id, follow_selection)
+        # Activate moved preset in controller at the very end (deferred to avoid re-entrancy)
+        QTimer.singleShot(0, lambda mid=moved_id: self._finalize_select_after_drop(mid))
+
+    def _finalize_select_after_drop(self, moved_role_id: str):
+        """
+        Final activation of the moved preset in controller after DnD completed and view got refreshed.
+        This is intentionally deferred to the next event loop tick.
+        """
+        try:
+            pid = moved_role_id
+            if not pid:
+                ids = self._current_selected_ids()
+                pid = ids[0] if ids else ""
+            if pid:
+                self.window.controller.presets.select_by_id(pid)
+        except Exception:
+            pass
+
+    def _move_row(self, from_row: int, to_row: int):
+        """Move row programmatically; persist order and keep selection attached to the same item."""
+        if from_row == to_row:
+            return
+        moved_id = self._reorder_and_persist(from_row, to_row)
+        self._refresh_after_order_change(moved_id, follow_selection=False)
+
+    # ----------------------------
+    # Mouse / DnD events
+    # ----------------------------
+
+    def _mouse_event_point(self, event):
+        if hasattr(event, "position"):
+            try:
+                p = event.position()
+                if hasattr(p, "toPoint"):
+                    return p.toPoint()
+            except Exception:
+                pass
+        if hasattr(event, "pos"):
+            return event.pos()
+        return self.viewport().mapFromGlobal(QCursor.pos())
+
     def mousePressEvent(self, event):
+        if self._model_updating:
+            event.ignore()
+            return
         if event.button() == Qt.LeftButton:
-            index = self.indexAt(event.pos())
+            index = self.indexAt(self._mouse_event_point(event))
             if not index.isValid():
                 return
-            super().mousePressEvent(event)
+            if self._dnd_enabled:
+                sel_model = self.selectionModel()
+                self._press_backup_selection = list(sel_model.selectedIndexes())
+                self._press_backup_current = self.currentIndex()
+                self._dragged_was_selected = any(ix.row() == index.row() for ix in self._press_backup_selection or [])
+                super().mousePressEvent(event)
+                # Keep old selection (do not auto-select dragged item yet)
+                sel_model.clearSelection()
+                for i in self._press_backup_selection or []:
+                    sel_model.select(i, QItemSelectionModel.Select | QItemSelectionModel.Rows)
+                if self._press_backup_current and self._press_backup_current.isValid():
+                    self.setCurrentIndex(self._press_backup_current)
+                self._press_pos = self._mouse_event_point(event)
+                self._press_index = index
+                self._drag_selection_applied = False
+                event.accept()
+                return
+            else:
+                super().mousePressEvent(event)
         elif event.button() == Qt.RightButton:
-            index = self.indexAt(event.pos())
+            index = self.indexAt(self._mouse_event_point(event))
             if index.isValid():
                 sel_model = self.selectionModel()
+                # Save original IDs (before we temporarily select right-click row)
+                self._ctx_menu_original_ids = []
+                for ix in sel_model.selectedRows():
+                    pid = ix.data(self.ROLE_ID)
+                    if pid:
+                        self._ctx_menu_original_ids.append(pid)
+
                 self._backup_selection = list(sel_model.selectedIndexes())
                 sel_model.clearSelection()
                 sel_model.select(
@@ -213,10 +619,188 @@ class PresetList(BaseList):
         else:
             super().mousePressEvent(event)
 
-    def selectionCommand(self, index, event=None):
+    def mouseMoveEvent(self, event):
+        if self._model_updating:
+            return
+        if not self._dnd_enabled:
+            return super().mouseMoveEvent(event)
+        if self._press_index is None or self._press_pos is None:
+            return super().mouseMoveEvent(event)
+        if not (event.buttons() & Qt.LeftButton):
+            return super().mouseMoveEvent(event)
+
+        cur = self._mouse_event_point(event)
+        dist = (cur - self._press_pos).manhattanLength()
+        threshold = QApplication.startDragDistance()
+        if dist < threshold:
+            return
+
+        # Pin current.* at the top; prevent dragging it
+        if self._press_index.row() == 0 or bool(self._press_index.data(self.ROLE_IS_SPECIAL)):
+            return super().mouseMoveEvent(event)
+
+        # Exception rule: at the start of drag, select the dragged item (view-only to avoid re-entrancy)
+        if not self._drag_selection_applied:
+            try:
+                sel_model = self.selectionModel()
+                prev_unlocked = self.unlocked
+                self.unlocked = True
+                try:
+                    sel_model.clearSelection()
+                    sel_model.select(self._press_index, QItemSelectionModel.ClearAndSelect | QItemSelectionModel.Rows)
+                    self.setCurrentIndex(self._press_index)
+                finally:
+                    self.unlocked = prev_unlocked
+            except Exception:
+                pass
+            self._drag_selection_applied = True
+
+        self._dragging = True
+        self.setCursor(QCursor(Qt.ClosedHandCursor))
+        super().mouseMoveEvent(event)
+
+    def mouseReleaseEvent(self, event):
+        if self._model_updating:
+            event.ignore()
+            return
+        try:
+            if self._dnd_enabled and event.button() == Qt.LeftButton:
+                self.unsetCursor()
+                if not self._dragging:
+                    idx = self.indexAt(self._mouse_event_point(event))
+                    if idx.isValid():
+                        pid = idx.data(self.ROLE_ID)
+                        if pid:
+                            self.window.controller.presets.select_by_id(pid)
+                        else:
+                            self.setCurrentIndex(idx)
+                            self.window.controller.presets.select(idx.row())
+        finally:
+            self._press_pos = None
+            self._press_index = None
+            self._press_backup_selection = None
+            self._press_backup_current = None
+            self._dragging = False
+            self._dragged_was_selected = False
+            self._drag_selection_applied = False
+        super().mouseReleaseEvent(event)
+
+    def dragEnterEvent(self, event):
+        if self._model_updating:
+            event.ignore()
+            return
+        if not self._dnd_enabled:
+            return
+        event.setDropAction(Qt.MoveAction)
+        event.acceptProposedAction()
+
+    def dragLeaveEvent(self, event):
+        if self._model_updating:
+            event.ignore()
+            return
+        self.unsetCursor()
+        super().dragLeaveEvent(event)
+
+    def dragMoveEvent(self, event):
+        if self._model_updating:
+            event.ignore()
+            return
+        if not self._dnd_enabled:
+            return
+        event.setDropAction(Qt.MoveAction)
+
+        pos = self._mouse_event_point(event)
+        idx = self.indexAt(pos)
+        # Do not allow dropping into the pinned first row zone
+        if idx.isValid() and idx.row() == 0:
+            rect = self.visualRect(idx)
+            if pos.y() <= rect.center().y():
+                event.ignore()
+                return
+        event.acceptProposedAction()
+
+    def dropEvent(self, event):
         """
-        Selection command
-        :param index: Index
-        :param event: Event
+        Fully handle flat row-to-row move. Persist order and defer view rebuild to next event loop,
+        so Qt can finish DnD teardown (prevents temporary disappearance).
         """
+        if self._model_updating:
+            event.ignore()
+            return
+        if not self._dnd_enabled:
+            return super().dropEvent(event)
+
+        model = self.model()
+        if model is None:
+            event.ignore()
+            return
+
+        # Source row (from press index if available)
+        if self._press_index is not None and self._press_index.isValid():
+            from_row = self._press_index.row()
+        else:
+            cur = self.currentIndex()
+            from_row = cur.row() if cur.isValid() else -1
+
+        if from_row < 0:
+            event.ignore()
+            self.unsetCursor()
+            self._drag_selection_applied = False
+            return
+
+        # Target row
+        pos = self._mouse_event_point(event)
+        idx = self.indexAt(pos)
+        if not idx.isValid():
+            to_row = model.rowCount()  # append
+        else:
+            rect = self.visualRect(idx)
+            to_row = idx.row()
+            if pos.y() > rect.center().y():
+                to_row += 1
+
+        # Keep first row pinned
+        if to_row <= 1:
+            to_row = 1
+
+        # Adjust when moving down (Qt inserts before position)
+        if to_row > from_row:
+            to_row -= 1
+
+        moved_id = self._reorder_and_persist(from_row, to_row)
+
+        # Defer the heavy refresh to the next event loop tick
+        self._pending_after_drop = (moved_id, False)
+        QTimer.singleShot(0, self._apply_after_drop)
+
+        # Properly finalize DnD in Qt and exit without mutating the model here
+        event.setDropAction(Qt.MoveAction)
+        event.acceptProposedAction()
+        self.unsetCursor()
+        self._drag_selection_applied = False
+
+    # ----------------------------
+    # Legacy helper (not used in new path)
+    # ----------------------------
+
+    def _persist_current_model_order(self):
+        """Deprecated in favor of _reorder_and_persist; retained for backward compatibility if needed."""
+        model = self.model()
+        if model is None:
+            return
+        uuids = []
+        for i in range(model.rowCount()):
+            if i == 0:
+                continue
+            idx = model.index(i, 0)
+            u = idx.data(self.ROLE_UUID)
+            if u and isinstance(u, str):
+                uuids.append(u)
+        mode = self.window.core.config.get('mode')
+        self.window.controller.presets.persist_order_for_mode(mode, uuids)
+
+    def selectionCommand(self, index, event=None):
+        # Prevent selection changes while model is updating (guards against stale indexes)
+        if self._model_updating:
+            return QItemSelectionModel.NoUpdate
         return super().selectionCommand(index, event)