✨ (data table) hide change columns for non-numerical data

owid · Aug 20, 2024 · 12c14f3 · 12c14f3
1 parent 7aa5910
commit 12c14f3
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/packages/@ourworldindata/grapher/src/dataTable/DataTable.tsx b/packages/@ourworldindata/grapher/src/dataTable/DataTable.tsx
@@ -789,15 +789,18 @@ export class DataTable extends React.Component<{
         targetTimes: number[]
         targetTimeMode: TargetTimeMode
     }): DimensionColumn[] {
-        // Inject delta columns if we have start & end values to compare in the table.
-        // One column for absolute difference, another for % difference.
+        // Inject delta columns if the data is numerical and we have start & end
+        // values to compare in the table. One column for absolute difference,
+        // another for % difference.
         const deltaColumns: DimensionColumn[] = []
-        if (targetTimeMode === TargetTimeMode.range) {
-            const { tableDisplay = {} } = sourceColumn.display ?? {}
-            if (!tableDisplay.hideAbsoluteChange)
-                deltaColumns.push({ key: RangeValueKey.delta })
-            if (!tableDisplay.hideRelativeChange)
-                deltaColumns.push({ key: RangeValueKey.deltaRatio })
+        if (sourceColumn.hasNumberFormatting) {
+            if (targetTimeMode === TargetTimeMode.range) {
+                const { tableDisplay = {} } = sourceColumn.display ?? {}
+                if (!tableDisplay.hideAbsoluteChange)
+                    deltaColumns.push({ key: RangeValueKey.delta })
+                if (!tableDisplay.hideRelativeChange)
+                    deltaColumns.push({ key: RangeValueKey.deltaRatio })
+            }
         }
 
         const valueColumns = targetTimes.map((targetTime, index) => ({