Block drag and drop for ROOT items on the widget level, and drop the TRASH item type

novelwriter/core/item.py

@@ -408,6 +408,8 @@ class NWItem():
             self._type = value
         elif isItemType(value):
             self._type = nwItemType[value]
+        elif value == "TRASH":
+            self._type = nwItemType.ROOT
         else:
             logger.error("Unrecognised item type '%s'", value)
             self._type = nwItemType.NO_TYPE

novelwriter/core/project.py

@@ -158,7 +158,7 @@ class NWProject():
         if trashHandle is None:
             newItem = NWItem(self)
             newItem.setName(trConst(nwLabels.CLASS_NAME[nwItemClass.TRASH]))
-            newItem.setType(nwItemType.TRASH)
+            newItem.setType(nwItemType.ROOT)
             newItem.setClass(nwItemClass.TRASH)
             self.projTree.append(None, None, newItem)
             self.projTree.updateItemData(newItem.itemHandle)

novelwriter/core/tree.py

@@ -100,14 +100,13 @@ class NWTree():
             if nwItem.itemClass == nwItemClass.ARCHIVE:
                 logger.verbose("Item '%s' is the archive folder", str(tHandle))
                 self._archRoot = tHandle
-
-        if nwItem.itemType == nwItemType.TRASH:
-            if self._trashRoot is None:
-                logger.verbose("Item '%s' is the trash folder", str(tHandle))
-                self._trashRoot = tHandle
-            else:
-                logger.error("Only one trash folder allowed")
-                return False
+            elif nwItem.itemClass == nwItemClass.TRASH:
+                if self._trashRoot is None:
+                    logger.verbose("Item '%s' is the trash folder", str(tHandle))
+                    self._trashRoot = tHandle
+                else:
+                    logger.error("Only one trash folder allowed")
+                    return False
 
         self._projTree[tHandle] = nwItem
         self._treeOrder.append(tHandle)

novelwriter/enum.py

@@ -32,7 +32,6 @@ class nwItemType(Enum):
     ROOT    = 1
     FOLDER  = 2
     FILE    = 3
-    TRASH   = 4
 
 # END Enum nwItemType
 

novelwriter/gui/projtree.py

@@ -762,31 +762,18 @@ class GuiProjectTree(QTreeWidget):
         drop is allowed or not. Disallowed drops are cancelled.
         """
         sHandle = self.getSelectedHandle()
-        dIndex = self.indexAt(theEvent.pos())
-        if sHandle is None or not dIndex.isValid():
+        if sHandle is None:
             logger.error("Invalid drag and drop event")
             return
 
-        sItem = self._getTreeItem(sHandle)
-        dItem = self.itemFromIndex(dIndex)
-        dHandle = dItem.data(self.C_NAME, Qt.UserRole)
-        snItem = self.theProject.projTree[sHandle]
-        dnItem = self.theProject.projTree[dHandle]
-
-        if snItem.itemType == nwItemType.ROOT or dnItem is None:
-            logger.debug("Drag'n'drop of item '%s' not accepted", sHandle)
-            theEvent.ignore()
-            self.theParent.makeAlert(self.tr(
-                "The item cannot be moved to that location."
-            ), nwAlert.ERROR)
-            return
-
         logger.debug("Drag'n'drop of item '%s' accepted", sHandle)
 
         self.propagateCount(sHandle, 0)
         QTreeWidget.dropEvent(self, theEvent)
         self._postItemMove(sHandle)
 
+        # Record undo information
+        sItem = self._getTreeItem(sHandle)
         pItem = sItem.parent()
         pIndex = 0
         if pItem is not None:
@@ -895,8 +882,7 @@ class GuiProjectTree(QTreeWidget):
         self._treeMap[tHandle] = newItem
         if pHandle is None:
             if nwItem.itemType == nwItemType.ROOT:
-                self.addTopLevelItem(newItem)
-            elif nwItem.itemType == nwItemType.TRASH:
+                newItem.setFlags(newItem.flags() ^ Qt.ItemIsDragEnabled)
                 self.addTopLevelItem(newItem)
             else:
                 self.theParent.makeAlert(self.tr(

novelwriter/gui/theme.py

@@ -639,9 +639,6 @@ class GuiIcons:
                     iconName = "proj_scene"
             elif tLayout == nwItemLayout.NOTE:
                 iconName = "proj_note"
-        elif tType == nwItemType.TRASH:
-            iconName = nwLabels.CLASS_ICON[tClass]
-
         if iconName is None:
             return QIcon()