From 7647176173d3e4d28290650761370efeede53505 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Mon, 31 Oct 2022 13:37:07 +0100
Subject: [PATCH] hide thumbnail widget if drop is disabled

---
 .../tools/publisher/widgets/create_widget.py  |  2 +-
 .../publisher/widgets/thumbnail_widget.py     | 62 -------------------
 openpype/tools/publisher/widgets/widgets.py   |  4 +-
 3 files changed, 3 insertions(+), 65 deletions(-)

diff --git a/openpype/tools/publisher/widgets/create_widget.py b/openpype/tools/publisher/widgets/create_widget.py
index a57a8791a8..4540e70eb8 100644
--- a/openpype/tools/publisher/widgets/create_widget.py
+++ b/openpype/tools/publisher/widgets/create_widget.py
@@ -558,7 +558,7 @@ class CreateWidget(QtWidgets.QWidget):
             self._set_context_enabled(creator_item.create_allow_context_change)
             self._refresh_asset()
 
-        self._thumbnail_widget.set_drop_enabled(
+        self._thumbnail_widget.setVisible(
             creator_item.create_allow_thumbnail
         )
 
diff --git a/openpype/tools/publisher/widgets/thumbnail_widget.py b/openpype/tools/publisher/widgets/thumbnail_widget.py
index 48f40f7b5b..53152f488f 100644
--- a/openpype/tools/publisher/widgets/thumbnail_widget.py
+++ b/openpype/tools/publisher/widgets/thumbnail_widget.py
@@ -55,8 +55,6 @@ class ThumbnailWidget(QtWidgets.QWidget):
         self.overlay_color = overlay_color
         self._default_pix = default_pix
 
-        self._drop_enabled = True
-
         self._current_pixes = None
         self._cached_pix = None
 
@@ -87,10 +85,6 @@ class ThumbnailWidget(QtWidgets.QWidget):
         return None
 
     def dragEnterEvent(self, event):
-        if not self._drop_enabled:
-            event.ignore()
-            return
-
         filepath = self._get_filepath_from_event(event)
         if filepath:
             event.setDropAction(QtCore.Qt.CopyAction)
@@ -100,9 +94,6 @@ class ThumbnailWidget(QtWidgets.QWidget):
         event.accept()
 
     def dropEvent(self, event):
-        if not self._drop_enabled:
-            return
-
         filepath = self._get_filepath_from_event(event)
         if not filepath:
             return
@@ -116,13 +107,6 @@ class ThumbnailWidget(QtWidgets.QWidget):
                 CardMessageTypes.error
             )
 
-    def set_drop_enabled(self, enabled):
-        if self._drop_enabled is enabled:
-            return
-        self._drop_enabled = enabled
-        self._cached_pix = None
-        self.repaint()
-
     def set_adapted_to_hint(self, enabled):
         self._adapted_to_size = enabled
         if self._width is not None:
@@ -172,10 +156,6 @@ class ThumbnailWidget(QtWidgets.QWidget):
         self.repaint()
 
     def _get_current_pixes(self):
-        if not self._drop_enabled:
-            # TODO different image for disabled drop
-            return [self._default_pix]
-
         if self._current_pixes is None:
             return [self._default_pix]
         return self._current_pixes
@@ -273,53 +253,11 @@ class ThumbnailWidget(QtWidgets.QWidget):
             y_offset = (height_offset_part * idx) + pix_y_offset
             final_painter.drawPixmap(x_offset, y_offset, pix)
 
-        if not self._drop_enabled:
-            overlay = self._get_drop_disabled_overlay(rect_width, rect_height)
-            final_painter.drawPixmap(0, 0, overlay)
 
         final_painter.end()
 
         self._cached_pix = final_pix
 
-    def _get_drop_disabled_overlay(self, width, height):
-        min_size = min(width, height)
-        circle_size = int(min_size * 0.8)
-        pen_width = int(circle_size * 0.1)
-        if pen_width < 1:
-            pen_width = 1
-
-        x_offset = int((width - circle_size) / 2)
-        y_offset = int((height - circle_size) / 2)
-        half_size = int(circle_size / 2)
-        angle = math.radians(45)
-        line_offset_p = QtCore.QPoint(
-            half_size * math.cos(angle),
-            half_size * math.sin(angle)
-        )
-        overlay_pix = QtGui.QPixmap(width, height)
-        overlay_pix.fill(QtCore.Qt.transparent)
-
-        painter = QtGui.QPainter()
-        painter.begin(overlay_pix)
-        painter.setRenderHints(
-            painter.Antialiasing
-            | painter.SmoothPixmapTransform
-            | painter.HighQualityAntialiasing
-        )
-        painter.setBrush(QtCore.Qt.transparent)
-        pen = QtGui.QPen(self.overlay_color)
-        pen.setWidth(pen_width)
-        painter.setPen(pen)
-        rect = QtCore.QRect(x_offset, y_offset, circle_size, circle_size)
-        painter.drawEllipse(rect)
-        painter.drawLine(
-            rect.center() - line_offset_p,
-            rect.center() + line_offset_p
-        )
-        painter.end()
-
-        return overlay_pix
-
     def _get_pix_offset_size(self, width, height, image_count):
         if image_count == 1:
             return 0, 0
diff --git a/openpype/tools/publisher/widgets/widgets.py b/openpype/tools/publisher/widgets/widgets.py
index fb9bd761f4..9af9595a97 100644
--- a/openpype/tools/publisher/widgets/widgets.py
+++ b/openpype/tools/publisher/widgets/widgets.py
@@ -1674,7 +1674,7 @@ class SubsetAttributesWidget(QtWidgets.QWidget):
             instance_ids.append(None)
 
         if not instance_ids:
-            self._thumbnail_widget.set_drop_enabled(False)
+            self._thumbnail_widget.setVisible(False)
             self._thumbnail_widget.set_current_thumbnails(None)
             return
 
@@ -1687,5 +1687,5 @@ class SubsetAttributesWidget(QtWidgets.QWidget):
             if path:
                 thumbnail_paths.append(path)
 
-        self._thumbnail_widget.set_drop_enabled(True)
+        self._thumbnail_widget.setVisible(True)
         self._thumbnail_widget.set_current_thumbnails(thumbnail_paths)