From 16929687d35df62b54fe5663987168b1a2725466 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 231e583..c0df0aa 100644
--- a/src/wireviz/Harness.py
+++ b/src/wireviz/Harness.py
@@ -158,9 +158,11 @@ class Harness:
             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: