From 24cb024dd9d27e5c68fcd6f7d9097297031d7845 Mon Sep 17 00:00:00 2001
From: Felix Wang <icyvapor@gmail.com>
Date: Wed, 8 Jun 2022 00:45:15 -0700
Subject: [PATCH] Maya: Look assigner UI improvements (#3208)

Co-authored-by: felix.wang <felix.wang@igg.com>
---
 openpype/hosts/maya/api/lib.py     |  5 ++++-
 openpype/tools/utils/host_tools.py | 10 ++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/openpype/hosts/maya/api/lib.py b/openpype/hosts/maya/api/lib.py
index 088304ab05a..bce03a648b1 100644
--- a/openpype/hosts/maya/api/lib.py
+++ b/openpype/hosts/maya/api/lib.py
@@ -1737,8 +1737,11 @@ def apply_shaders(relationships, shadernodes, nodes):
             log.warning("No nodes found for shading engine "
                         "'{0}'".format(id_shading_engines[0]))
             continue
+        try:
+            cmds.sets(filtered_nodes, forceElement=id_shading_engines[0])
+        except RuntimeError as rte:
+            log.error("Error during shader assignment: {}".format(rte))
 
-        cmds.sets(filtered_nodes, forceElement=id_shading_engines[0])
     # endregion
 
     apply_attributes(attributes, nodes_by_id)
diff --git a/openpype/tools/utils/host_tools.py b/openpype/tools/utils/host_tools.py
index d8f45701201..9dbbe25fdac 100644
--- a/openpype/tools/utils/host_tools.py
+++ b/openpype/tools/utils/host_tools.py
@@ -105,6 +105,7 @@ def show_loader(self, parent=None, use_context=None):
             loader_tool.show()
             loader_tool.raise_()
             loader_tool.activateWindow()
+            loader_tool.showNormal()
 
             if use_context is None:
                 use_context = False
@@ -180,6 +181,7 @@ def show_scene_inventory(self, parent=None):
             # Pull window to the front.
             scene_inventory_tool.raise_()
             scene_inventory_tool.activateWindow()
+            scene_inventory_tool.showNormal()
 
     def get_library_loader_tool(self, parent):
         """Create, cache and return library loader tool window."""
@@ -200,8 +202,10 @@ def show_library_loader(self, parent=None):
             library_loader_tool.show()
             library_loader_tool.raise_()
             library_loader_tool.activateWindow()
+            library_loader_tool.showNormal()
             library_loader_tool.refresh()
 
+
     def show_publish(self, parent=None):
         """Try showing the most desirable publish GUI
 
@@ -243,6 +247,11 @@ def show_look_assigner(self, parent=None):
             look_assigner_tool = self.get_look_assigner_tool(parent)
             look_assigner_tool.show()
 
+            # Pull window to the front.
+            look_assigner_tool.raise_()
+            look_assigner_tool.activateWindow()
+            look_assigner_tool.showNormal()
+
     def get_experimental_tools_dialog(self, parent=None):
         """Dialog of experimental tools.
 
@@ -270,6 +279,7 @@ def show_experimental_tools_dialog(self, parent=None):
             dialog.show()
             dialog.raise_()
             dialog.activateWindow()
+            dialog.showNormal()
 
     def get_tool_by_name(self, tool_name, parent=None, *args, **kwargs):
         """Show tool by it's name.