From b0702ceff50dfb08d7464cad83ea598ab76413d0 Mon Sep 17 00:00:00 2001 From: Pete Richards Date: Tue, 17 Jul 2018 13:55:35 -0700 Subject: [PATCH] [Table] Check alarms for all values (#2115) Check alarm status for all values, not just ranges. This allows limit evaluators to apply styling to additional columns if desired by developers. --- platform/features/table/src/TableColumn.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/platform/features/table/src/TableColumn.js b/platform/features/table/src/TableColumn.js index acb9b9340d..971fd0e9ba 100644 --- a/platform/features/table/src/TableColumn.js +++ b/platform/features/table/src/TableColumn.js @@ -50,9 +50,7 @@ define(function () { }; TableColumn.prototype.getValue = function (telemetryDatum, limitEvaluator) { - var isValueColumn = !!(this.metadatum.hints.y || this.metadatum.hints.range); - var alarm = isValueColumn && - limitEvaluator && + var alarm = limitEvaluator && limitEvaluator.evaluate(telemetryDatum, this.metadatum); var value = { text: this.formatter.format(telemetryDatum),