fix: auto adjust scale min to zero for column/bar (#2012)

* fix: auto adjust scale min to zero for column/bar

* Update y-meta-min-max-spec.ts

* Update y-meta-min-max-spec.ts

Co-authored-by: hustcc <[email protected]>

Author: lessmost

__tests__/unit/plots/bar/y-meta-min-max-spec.ts

@@ -0,0 +1,37 @@
+import { Bar } from '../../../../src';
+import { createDiv } from '../../../utils/dom';
+
+describe('bar', () => {
+  it('y min, max meta', () => {
+    const plot = new Bar(createDiv(), {
+      width: 400,
+      height: 300,
+      yField: 'date',
+      xField: 'value',
+      data: [
+        { date: 'a', value: 10 },
+        { date: 'b', value: 20 },
+      ],
+    });
+
+    plot.render();
+    expect(plot.chart.getScaleByField('value').min).toBe(0);
+    expect(plot.chart.getScaleByField('value').max).toBe(20);
+
+    plot.update({
+      width: 400,
+      height: 300,
+      yField: 'date',
+      xField: 'value',
+      data: [
+        { date: 'a', value: -10 },
+        { date: 'b', value: -20 },
+      ],
+    });
+
+    expect(plot.chart.getScaleByField('value').min).toBe(-20);
+    expect(plot.chart.getScaleByField('value').max).toBe(0);
+
+    plot.destroy();
+  });
+});

__tests__/unit/plots/column/y-meta-min-max-spec.ts

@@ -0,0 +1,37 @@
+import { Column } from '../../../../src';
+import { createDiv } from '../../../utils/dom';
+
+describe('column', () => {
+  it('y min, max meta', () => {
+    const plot = new Column(createDiv(), {
+      width: 400,
+      height: 300,
+      xField: 'date',
+      yField: 'value',
+      data: [
+        { date: 'a', value: 10 },
+        { date: 'b', value: 20 },
+      ],
+    });
+
+    plot.render();
+    expect(plot.chart.getScaleByField('value').min).toBe(0);
+    expect(plot.chart.getScaleByField('value').max).toBe(20);
+
+    plot.update({
+      width: 400,
+      height: 300,
+      xField: 'date',
+      yField: 'value',
+      data: [
+        { date: 'a', value: -10 },
+        { date: 'b', value: -20 },
+      ],
+    });
+
+    expect(plot.chart.getScaleByField('value').min).toBe(-20);
+    expect(plot.chart.getScaleByField('value').max).toBe(0);
+
+    plot.destroy();
+  });
+});

src/plots/column/adaptor.ts

@@ -15,6 +15,7 @@ import { conversionTag } from '../../adaptor/conversion-tag';
 import { connectedArea } from '../../adaptor/connected-area';
 import { interval } from '../../adaptor/geometries';
 import { flow, transformLabel, deepAssign } from '../../utils';
+import { adjustYMetaByZero } from '../../utils/data';
 import { percent } from '../../utils/transform/percent';
 import { Datum } from '../../types';
 import { ColumnOptions } from './types';
@@ -88,7 +89,7 @@ function geometry(params: Params<ColumnOptions>): Params<ColumnOptions> {
  */
 function meta(params: Params<ColumnOptions>): Params<ColumnOptions> {
   const { options } = params;
-  const { xAxis, yAxis, xField, yField } = options;
+  const { xAxis, yAxis, xField, yField, data } = options;
 
   return flow(
     scale(
@@ -100,6 +101,7 @@ function meta(params: Params<ColumnOptions>): Params<ColumnOptions> {
         [xField]: {
           type: 'cat',
         },
+        [yField]: adjustYMetaByZero(data, yField),
       }
     )
   )(params);