move color resolution to dataBar options

Author: gzh2003

packages/grid/src/DataBarCellRenderer.ts

@@ -147,20 +147,7 @@ class DataBarCellRenderer extends CellRenderer {
 
     context.save();
     context.textAlign = textAlign;
-
-    // Use explicit text color if set.
-    // Otherwise, fall back to the databar color for text.
-    if (textColor != null) {
-      context.fillStyle = textColor;
-    } else if (hasGradient) {
-      const color =
-        value >= 0 ? dataBarColor[dataBarColor.length - 1] : dataBarColor[0];
-      context.fillStyle = color;
-    } else {
-      context.fillStyle = Array.isArray(dataBarColor)
-        ? dataBarColor[0]
-        : dataBarColor;
-    }
+    context.fillStyle = textColor;
     context.textBaseline = 'middle';
     context.font = theme.font;
 

packages/grid/src/DataBarGridModel.ts

@@ -35,7 +35,7 @@ export interface DataBarOptions {
   columnMax: number;
   axis: AxisOption;
   color: GridColor | GridColor[];
-  textColor?: GridColor;
+  textColor: GridColor;
   valuePlacement: ValuePlacementOption;
   opacity: number;
   markers: Marker[];

packages/grid/src/MockDataBarGridModel.ts

@@ -143,11 +143,23 @@ class MockDataBarGridModel extends GridModel implements DataBarGridModel {
     const color = value >= 0 ? positiveColor : negativeColor;
     const markers = this.markers.get(column) ?? [];
 
+    const hasGradient = Array.isArray(color) && color.length > 1;
+    let textColor: string;
+    if (hasGradient) {
+      textColor = value >= 0 ? color[color.length - 1] : color[0];
+    } else if (Array.isArray(color)) {
+      const [first] = color;
+      textColor = first;
+    } else {
+      textColor = color;
+    }
+
     return {
       columnMin,
       columnMax,
       axis,
       color,
+      textColor,
       valuePlacement,
       opacity,
       markers,