From c7959ff06e1151fa1b0b99e916e55f5a21340cbe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torkel=20=C3=96degaard?= Date: Thu, 10 Aug 2017 14:12:31 +0200 Subject: [PATCH] fix: Using table cell links and ad hoc filters together now works & looks correct, fixes #8052 --- public/app/plugins/panel/table/renderer.ts | 12 ++++++------ public/sass/components/_panel_table.scss | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/public/app/plugins/panel/table/renderer.ts b/public/app/plugins/panel/table/renderer.ts index 02f0ceca72c..cab3f698ae6 100644 --- a/public/app/plugins/panel/table/renderer.ts +++ b/public/app/plugins/panel/table/renderer.ts @@ -157,9 +157,9 @@ export class TableRenderer { // because of the fixed table headers css only solution // there is an issue if header cell is wider the cell // this hack adds header content to cell (not visible) - var widthHack = ''; + var columnHtml = ''; if (addWidthHack) { - widthHack = '
' + this.table.columns[columnIndex].title + '
'; + columnHtml = '
' + this.table.columns[columnIndex].title + '
'; } if (value === undefined) { @@ -173,8 +173,6 @@ export class TableRenderer { cellClasses.push("table-panel-cell-pre"); } - var columnHtml = widthHack + value; - if (column.style && column.style.link) { // Render cell as link var scopedVars = this.renderRowVariables(rowIndex); @@ -185,11 +183,13 @@ export class TableRenderer { var cellTarget = column.style.linkTargetBlank ? '_blank' : ''; cellClasses.push("table-panel-cell-link"); - columnHtml = ` + columnHtml += ` - ${columnHtml} + ${value} `; + } else { + columnHtml += value; } if (column.filterable) { diff --git a/public/sass/components/_panel_table.scss b/public/sass/components/_panel_table.scss index cf81dca4465..77652bc634d 100644 --- a/public/sass/components/_panel_table.scss +++ b/public/sass/components/_panel_table.scss @@ -84,7 +84,7 @@ a { padding: 0.45em 0 0.45em 1.1em; height: 100%; - width: 100%; + display: inline-block; } }