warehack.ing/WireViz
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Resolve edge case of empty HTML tables

Browse Source
This commit is contained in:
Daniel Rojas 2021-09-23 16:20:59 +02:00
parent 8ea8248721
commit 2a62dae9ee
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/wireviz/wv_gv_html.py
View File

@ -14,6 +14,8 @@ def nested_html_table(rows: List[Union[str, List[Optional[str]], None]], table_a
# attributes in any leading <tdX> inside a list are injected into to the preceeding <td> tag # attributes in any leading <tdX> inside a list are injected into to the preceeding <td> tag
html = [] html = []
html.append(f'<table border="0" cellspacing="0" cellpadding="0"{table_attrs or ""}>') html.append(f'<table border="0" cellspacing="0" cellpadding="0"{table_attrs or ""}>')
num_rows = 0
for row in rows: for row in rows:
if isinstance(row, List): if isinstance(row, List):
if len(row) > 0 and any(row): if len(row) > 0 and any(row):
@ -25,10 +27,14 @@ def nested_html_table(rows: List[Union[str, List[Optional[str]], None]], table_a
html.append(f' <td balign="left">{cell}</td>'.replace('><tdX', '')) html.append(f' <td balign="left">{cell}</td>'.replace('><tdX', ''))
html.append(' </tr></table>') html.append(' </tr></table>')
html.append(' </td></tr>') html.append(' </td></tr>')
num_rows = num_rows + 1
elif row is not None: elif row is not None:
html.append(' <tr><td>') html.append(' <tr><td>')
html.append(f' {row}') html.append(f' {row}')
html.append(' </td></tr>') html.append(' </td></tr>')
num_rows = num_rows + 1
if num_rows == 0: # empty table
html.append('<tr><td></td></tr>') # generate empty cell to avoid GraphViz errors
html.append('</table>') html.append('</table>')
return html return html