[8.0] [XY] Defaults the point size to 1 and corrects the vislib migrations (#118994)

src/plugins/vis_types/xy/public/utils/get_series_params.test.ts

@@ -45,7 +45,7 @@ describe('getSeriesParams', () => {
     );
     expect(seriesParams).toStrictEqual([
       {
-        circlesRadius: 3,
+        circlesRadius: 1,
         data: {
           id: '1',
           label: 'Total quantity',

src/plugins/vis_types/xy/public/utils/get_series_params.ts

@@ -22,7 +22,7 @@ const makeSerie = (
     type: ChartType.Line,
     drawLinesBetweenPoints: true,
     showCircles: true,
-    circlesRadius: 3,
+    circlesRadius: 1,
     interpolate: InterpolationMode.Linear,
     lineWidth: 2,
     valueAxis: defaultValueAxis,

src/plugins/vis_types/xy/public/vis_types/area.ts

@@ -95,7 +95,7 @@ export const areaVisTypeDefinition = {
           drawLinesBetweenPoints: true,
           lineWidth: 2,
           showCircles: true,
-          circlesRadius: 3,
+          circlesRadius: 1,
           interpolate: InterpolationMode.Linear,
           valueAxis: 'ValueAxis-1',
         },

src/plugins/vis_types/xy/public/vis_types/histogram.ts

@@ -99,7 +99,7 @@ export const histogramVisTypeDefinition = {
           drawLinesBetweenPoints: true,
           lineWidth: 2,
           showCircles: true,
-          circlesRadius: 3,
+          circlesRadius: 1,
         },
       ],
       radiusRatio: 0,

src/plugins/vis_types/xy/public/vis_types/horizontal_bar.ts

@@ -100,7 +100,7 @@ export const horizontalBarVisTypeDefinition = {
           drawLinesBetweenPoints: true,
           lineWidth: 2,
           showCircles: true,
-          circlesRadius: 3,
+          circlesRadius: 1,
         },
       ],
       addTooltip: true,

src/plugins/vis_types/xy/public/vis_types/line.ts

@@ -97,7 +97,7 @@ export const lineVisTypeDefinition = {
           lineWidth: 2,
           interpolate: InterpolationMode.Linear,
           showCircles: true,
-          circlesRadius: 3,
+          circlesRadius: 1,
         },
       ],
       addTooltip: true,

src/plugins/visualizations/server/migrations/visualization_saved_object_migrations.test.ts

@@ -1674,7 +1674,8 @@ describe('migration visualization', () => {
       type = 'area',
       categoryAxes?: object[],
       valueAxes?: object[],
-      hasPalette = false
+      hasPalette = false,
+      hasCirclesRadius = false
     ) => ({
       attributes: {
         title: 'My Vis',
@@ -1694,6 +1695,21 @@ describe('migration visualization', () => {
                 labels: {},
               },
             ],
+            seriesParams: [
+              {
+                show: true,
+                type,
+                mode: 'stacked',
+                drawLinesBetweenPoints: true,
+                lineWidth: 2,
+                showCircles: true,
+                interpolate: 'linear',
+                valueAxis: 'ValueAxis-1',
+                ...(hasCirclesRadius && {
+                  circlesRadius: 3,
+                }),
+              },
+            ],
             ...(hasPalette && {
               palette: {
                 type: 'palette',
@@ -1732,6 +1748,20 @@ describe('migration visualization', () => {
       expect(palette.name).toEqual('default');
     });
 
+    it("should decorate existing docs with the circlesRadius attribute if it doesn't exist", () => {
+      const migratedTestDoc = migrate(getTestDoc());
+      const [result] = JSON.parse(migratedTestDoc.attributes.visState).params.seriesParams;
+
+      expect(result.circlesRadius).toEqual(1);
+    });
+
+    it('should not decorate existing docs with the circlesRadius attribute if it exists', () => {
+      const migratedTestDoc = migrate(getTestDoc('area', undefined, undefined, true, true));
+      const [result] = JSON.parse(migratedTestDoc.attributes.visState).params.seriesParams;
+
+      expect(result.circlesRadius).toEqual(3);
+    });
+
     describe('labels.filter', () => {
       it('should keep existing categoryAxes labels.filter value', () => {
         const migratedTestDoc = migrate(getTestDoc('area', [{ labels: { filter: false } }]));

src/plugins/visualizations/server/migrations/visualization_saved_object_migrations.ts

@@ -867,6 +867,20 @@ const decorateAxes = <T extends { labels: { filter?: boolean } }>(
     },
   }));
 
+/**
+ * Defaults circlesRadius to 1 if it is not configured
+ */
+const addCirclesRadius = <T extends { circlesRadius: number }>(axes: T[]): T[] =>
+  axes.map((axis) => {
+    const hasCircleRadiusAttribute = Number.isFinite(axis?.circlesRadius);
+    return {
+      ...axis,
+      ...(!hasCircleRadiusAttribute && {
+        circlesRadius: 1,
+      }),
+    };
+  });
+
 // Inlined from vis_type_xy
 const CHART_TYPE_AREA = 'area';
 const CHART_TYPE_LINE = 'line';
@@ -913,10 +927,12 @@ const migrateVislibAreaLineBarTypes: SavedObjectMigrationFn<any, any> = (doc) =>
               valueAxes:
                 visState.params.valueAxes &&
                 decorateAxes(visState.params.valueAxes, isHorizontalBar),
+              seriesParams:
+                visState.params.seriesParams && addCirclesRadius(visState.params.seriesParams),
               isVislibVis: true,
               detailedTooltip: true,
               ...(isLineOrArea && {
-                fittingFunction: 'zero',
+                fittingFunction: 'linear',
               }),
             },
           }),