diff --git a/turnstone/console/static/governance.js b/turnstone/console/static/governance.js index 2896672b..664a6eb3 100644 --- a/turnstone/console/static/governance.js +++ b/turnstone/console/static/governance.js @@ -688,13 +688,6 @@ function _renderGovSkills(items) { var html = ""; for (var i = 0; i < items.length; i++) { var t = items[i]; - var vars = ""; - try { - var vlist = JSON.parse(t.variables || "[]"); - vars = vlist.join(", "); - } catch (e) { - vars = t.variables; - } var activationBadge = ""; var activation = t.activation || "named"; if (activation === "default") { @@ -715,7 +708,8 @@ function _renderGovSkills(items) { : ""; var catBadge = '' + escapeHtml(t.category) + ""; - var scanBadge = ""; + // Build risk column content with tooltip + var riskCell = ""; if (t.scan_status) { var scanClass = { @@ -725,12 +719,45 @@ function _renderGovSkills(items) { high: "scope-scan-high", critical: "scope-scan-critical", }[t.scan_status] || ""; - scanBadge = - ' ' + + '" tabindex="0" role="button" aria-label="Risk: ' + escapeHtml(t.scan_status) + + (tipParts.length ? ". " + escapeHtml(tipParts.join(". ")) : "") + + '" title="' + + escapeHtml(tipText) + + '">' + + escapeHtml(scanIcon + t.scan_status) + ""; + } else { + riskCell = + '\u2014'; } var resBadge = ""; if (t.resource_count > 0) { @@ -745,13 +772,15 @@ function _renderGovSkills(items) { var deleteDisabled = t.readonly ? " disabled" : ""; html += '
' + + '' + + catBadge + + "" + '' + escapeHtml(t.name) + " " + activationBadge + defBadge + originBadge + - scanBadge + resBadge + (t.description ? '
' + @@ -759,12 +788,9 @@ function _renderGovSkills(items) { "" : "") + "
" + - '' + - catBadge + + '' + + riskCell + "" + - '' + - escapeHtml(vars || "\u2014") + - "" + '' + '