ui: Fix nesting of slice argument table
This patch makes the arguments of a slice be displayed directly in the
"Slice Details" table rather than as standalone <tr> <td> tags (with no
containing <table>). We also remove the "Args" heading since looks a bit
odd being an extra row in the table rather than a real heading (which
would take more space).
Change-Id: I2217e8b64b38a368478d1acedb10baf6224ec7e0
diff --git a/ui/src/frontend/chrome_slice_panel.ts b/ui/src/frontend/chrome_slice_panel.ts
index c1bc604..a0a7f35 100644
--- a/ui/src/frontend/chrome_slice_panel.ts
+++ b/ui/src/frontend/chrome_slice_panel.ts
@@ -48,8 +48,8 @@
toNs(sliceInfo.dur) === -1 ?
'-1 (Did not end)' :
timeToCode(sliceInfo.dur)}`)),
- this.getDescription(sliceInfo.description)),
- this.getArgs(sliceInfo.args),
+ this.getDescription(sliceInfo.description),
+ this.getArgs(sliceInfo.args)),
));
} else {
return m(
@@ -66,7 +66,7 @@
getArgs(args?: Map<string, string>): m.Vnode[] {
if (!args || args.size === 0) return [];
- const result = [m('tr', m('th', 'Args'))];
+ const result = [];
for (const [key, value] of args) {
result.push(m('tr', m('th', key), m('td', value)));
}
