From a6d5316b356bdfb63fd49848550c2e4a2685e69c Mon Sep 17 00:00:00 2001
From: Liam Hoflay <Liametc@users.noreply.github.com>
Date: Mon, 4 Nov 2019 10:48:25 +0000
Subject: [PATCH] Feature/summary display (#6)

* Adding better formatting to publish dialog for summaries.

* Doc string added
---
 python/tk_multi_publish2/dialog.py            |  1 +
 .../publish_tree_widget/tree_node_item.py     | 20 ++++++++++++++-----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/python/tk_multi_publish2/dialog.py b/python/tk_multi_publish2/dialog.py
index 6b8529ee..d727f56d 100644
--- a/python/tk_multi_publish2/dialog.py
+++ b/python/tk_multi_publish2/dialog.py
@@ -696,6 +696,7 @@ def _create_item_details(self, tree_item):
 
         else:
             summary_text = "<p>The following items will be processed:</p>"
+            summary_text += "<style>ul { margin-left:-20px; }</style>"
             summary_text += "".join(["<p>%s</p>" % line for line in summary])
 
         self.ui.item_summary.setText(summary_text)
diff --git a/python/tk_multi_publish2/publish_tree_widget/tree_node_item.py b/python/tk_multi_publish2/publish_tree_widget/tree_node_item.py
index 5a0f4f44..d48be5d0 100644
--- a/python/tk_multi_publish2/publish_tree_widget/tree_node_item.py
+++ b/python/tk_multi_publish2/publish_tree_widget/tree_node_item.py
@@ -73,10 +73,13 @@ def __repr__(self):
     def __str__(self):
         return "%s %s" % (self._item.type_display, self._item.name)
 
-    def create_summary(self):
+    def create_summary(self, level=0):
         """
         Creates summary of actions
 
+        :param level: Indentation level of this item lives within the tree.
+        :type level: int
+
         :returns: List of strings
         """
         if self.checked:
@@ -91,16 +94,23 @@ def create_summary(self):
                     task_summaries.extend(child_item.create_summary())
                 else:
                     # sub-items
-                    items_summaries.extend(child_item.create_summary())
+                    items_summaries.extend(child_item.create_summary(level + 1))
 
             summary = []
 
             if len(task_summaries) > 0:
 
-                summary_str = "<b>%s</b><br>" % self.item.name
-                summary_str += "<br>".join(
-                    ["&ndash; %s" % task_summary for task_summary in task_summaries]
+                summary_str = "<ul><li>" * level
+                summary_str += "<b>%s</b>" % self.item.name
+                summary_str += '<ul style="list-style-type:circle;">'
+                summary_str += "".join(
+                    [
+                        "<li><i>%s</i></li>" % task_summary
+                        for task_summary in task_summaries
+                    ]
                 )
+                summary_str += "</ul>"
+                summary_str += "</li></ul>" * level
                 summary.append(summary_str)
 
             summary.extend(items_summaries)