From 093b04a1189634257a7372f534eb45de615f7340 Mon Sep 17 00:00:00 2001
From: Daniel Rojas <github@danielrojas.net>
Date: Thu, 2 Jul 2020 19:09:27 +0200
Subject: [PATCH] Fine-tune cable appearance

Add hack to make manufacturer / part number row have vertical separators, to be visually consistent with connectors.

In the future, connectors will have to be modified to be HTML tables instead of GraphViz record nodes, in order to support colored bands for the connector, or individual pins (#53). When this is implemented, a cleaner solution should be found for both connectors and cables.
---
 src/wireviz/Harness.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/wireviz/Harness.py b/src/wireviz/Harness.py
index 231e583d..c0df0aa9 100644
--- a/src/wireviz/Harness.py
+++ b/src/wireviz/Harness.py
@@ -158,9 +158,11 @@ def create_graph(self):
             if cable.show_name:
                 html = f'{html}<tr><td colspan="{len(attributes)}">{cable.name}</td></tr>'
             if(len(identification) > 0):  # print an identification row if values specified
-                html = f'{html}<tr><td colspan="{len(attributes)}"><table border="0" cellspacing="0" cellpadding="0" cellborder="0"><tr>'
-                for attrib in identification:
-                    html = f'{html}<td>{attrib}</td>'
+                html = f'{html}<tr><td colspan="{len(attributes)}" cellpadding="0"><table border="0" cellspacing="0" cellpadding="3" cellborder="1"><tr>'
+                for attrib in identification[0:-1]:
+                    html = f'{html}<td sides="R">{attrib}</td>' # all columns except last have a border on the right (sides="R")
+                if len(identification) > 0:
+                    html = f'{html}<td border="0">{identification[-1]}</td>' # last column has no border on the right because the enclosing table borders it
                 html = f'{html}</tr></table></td></tr>'  # end identification row
             html = f'{html}<tr>'  # attribute row
             for attrib in attributes: