From 03a207de92ea0758d291fcde1c2e7705424d66be Mon Sep 17 00:00:00 2001
From: Daniel Rojas <github@danielrojas.net>
Date: Wed, 11 Nov 2020 15:00:32 +0100
Subject: [PATCH] Add more suggestions by @kvid

---
 src/wireviz/DataClasses.py | 2 +-
 src/wireviz/Harness.py     | 8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/wireviz/DataClasses.py b/src/wireviz/DataClasses.py
index a138064..bdc5be9 100644
--- a/src/wireviz/DataClasses.py
+++ b/src/wireviz/DataClasses.py
@@ -125,7 +125,7 @@ class Connector:
                 raise Exception('You need to specify at least one, pincount, pins or pinlabels')
             self.pincount = max(len(self.pins), len(self.pinlabels), len(self.pincolors))
 
-        # create default lists for pins (sequential) and pinlabels (blank) if not specified
+        # create default list for pins (sequential) if not specified
         if not self.pins:
             self.pins = list(range(1, self.pincount + 1))
 
diff --git a/src/wireviz/Harness.py b/src/wireviz/Harness.py
index bbc68b7..be8cf1a 100644
--- a/src/wireviz/Harness.py
+++ b/src/wireviz/Harness.py
@@ -122,9 +122,13 @@ class Harness:
                     if connector.pincolors:
                         if pincolor in wv_colors._color_hex.keys():
                             pinhtml.append(f'    <td sides="tbl">{pincolor}</td>')
-                            pinhtml.append(f'    <td sides="tbr"><table border="0" cellborder="1"><tr><td bgcolor="{wv_colors.translate_color(pincolor, "HEX")}" width="8" height="8" fixedsize="true"></td></tr></table></td>')
+                            pinhtml.append( '    <td sides="tbr">')
+                            pinhtml.append( '     <table border="0" cellborder="1"><tr>')
+                            pinhtml.append(f'      <td bgcolor="{wv_colors.translate_color(pincolor, "HEX")}" width="8" height="8" fixedsize="true"></td>')
+                            pinhtml.append( '     </tr></table>')
+                            pinhtml.append( '    </td>')
                         else:
-                            pinhtml.append(f'    <td colspan="2"></td>')
+                            pinhtml.append( '    <td colspan="2"></td>')
 
                     if connector.ports_right:
                         pinhtml.append(f'    <td port="p{pin}r">{pin}</td>')