[Vislib] Removes old implementation of xy chart

docs/management/advanced-options.asciidoc

@@ -529,10 +529,6 @@ of the chart. Use numbers between 0 and 1. The lower the number, the more the hi
 [[visualization-heatmap-maxbuckets]]`visualization:heatmap:maxBuckets`::
 The maximum number of buckets a datasource can return. High numbers can have a negative impact on your browser rendering performance.
 
-[[visualization-visualize-chartslibrary]]`visualization:visualize:legacyChartsLibrary`::
-**The legacy XY charts are deprecated and will not be supported as of 7.16.**
-The visualize editor uses a new XY charts library with improved performance, color palettes, fill capacity, and more. Enable this option if you prefer to use the legacy charts library.
-
 [[visualization-visualize-pieChartslibrary]]`visualization:visualize:legacyPieChartsLibrary`::
 The visualize editor uses new pie charts with improved performance, color palettes, label positioning, and more. Enable this option if you prefer to use to the legacy charts library.
 

src/plugins/kibana_usage_collection/server/collectors/management/schema.ts

@@ -400,10 +400,6 @@ export const stackManagementSchema: MakeSchemaFrom<UsageStats> = {
     type: 'boolean',
     _meta: { description: 'Non-default value of setting.' },
   },
-  'visualization:visualize:legacyChartsLibrary': {
-    type: 'boolean',
-    _meta: { description: 'Non-default value of setting.' },
-  },
   'visualization:visualize:legacyPieChartsLibrary': {
     type: 'boolean',
     _meta: { description: 'Non-default value of setting.' },

src/plugins/kibana_usage_collection/server/collectors/management/types.ts

@@ -27,7 +27,6 @@ export interface UsageStats {
   'autocomplete:useTimeRange': boolean;
   'autocomplete:valueSuggestionMethod': string;
   'search:timeout': number;
-  'visualization:visualize:legacyChartsLibrary': boolean;
   'visualization:visualize:legacyPieChartsLibrary': boolean;
   'doc_table:legacy': boolean;
   'discover:modifyColumnsOnSwitch': boolean;

src/plugins/telemetry/schema/oss_plugins.json

@@ -7616,12 +7616,6 @@
             "description": "Non-default value of setting."
           }
         },
-        "visualization:visualize:legacyChartsLibrary": {
-          "type": "boolean",
-          "_meta": {
-            "description": "Non-default value of setting."
-          }
-        },
         "visualization:visualize:legacyPieChartsLibrary": {
           "type": "boolean",
           "_meta": {

src/plugins/vis_type_timelion/public/components/timelion_vis_component.tsx

@@ -177,7 +177,7 @@ const TimelionVisComponent = ({
   }, [chart]);
 
   return (
-    <div className="timelionChart" data-test-subj="visTypeXyChart">
+    <div className="timelionChart" data-test-subj="timelionChart">
       {title && (
         <EuiTitle className="timelionChart__topTitle" size="xxxs">
           <h4>{title}</h4>

src/plugins/vis_types/vislib/public/plugin.ts

@@ -13,16 +13,11 @@ import { VisualizationsSetup } from '../../../visualizations/public';
 import { ChartsPluginSetup } from '../../../charts/public';
 import { DataPublicPluginStart } from '../../../data/public';
 import { KibanaLegacyStart } from '../../../kibana_legacy/public';
-import { LEGACY_CHARTS_LIBRARY } from '../../xy/common/index';
 import { LEGACY_PIE_CHARTS_LIBRARY } from '../../pie/common/index';
 
 import { createVisTypeVislibVisFn } from './vis_type_vislib_vis_fn';
 import { createPieVisFn } from './pie_fn';
-import {
-  convertedTypeDefinitions,
-  pieVisTypeDefinition,
-  visLibVisTypeDefinitions,
-} from './vis_type_vislib_vis_types';
+import { visLibVisTypeDefinitions, pieVisTypeDefinition } from './vis_type_vislib_vis_types';
 import { setFormatService, setDataActions } from './services';
 import { getVislibVisRenderer } from './vis_renderer';
 
@@ -51,11 +46,8 @@ export class VisTypeVislibPlugin
     core: VisTypeVislibCoreSetup,
     { expressions, visualizations, charts }: VisTypeVislibPluginSetupDependencies
   ) {
-    const typeDefinitions = !core.uiSettings.get(LEGACY_CHARTS_LIBRARY, false)
-      ? convertedTypeDefinitions
-      : visLibVisTypeDefinitions;
     // register vislib XY axis charts
-    typeDefinitions.forEach(visualizations.createBaseVisualization);
+    visLibVisTypeDefinitions.forEach(visualizations.createBaseVisualization);
     expressions.registerRenderer(getVislibVisRenderer(core, charts));
     expressions.registerFunction(createVisTypeVislibVisFn());
 

src/plugins/vis_types/vislib/public/types.ts

@@ -37,12 +37,7 @@ export const GaugeType = Object.freeze({
 export type GaugeType = $Values<typeof GaugeType>;
 
 export const VislibChartType = Object.freeze({
-  Histogram: 'histogram' as const,
-  HorizontalBar: 'horizontal_bar' as const,
-  Line: 'line' as const,
   Pie: 'pie' as const,
-  Area: 'area' as const,
-  PointSeries: 'point_series' as const,
   Heatmap: 'heatmap' as const,
   Gauge: 'gauge' as const,
   Goal: 'goal' as const,

src/plugins/vis_types/vislib/public/vis_type_vislib_vis_types.ts

@@ -7,27 +7,13 @@
  */
 
 import { VisTypeDefinition } from 'src/plugins/visualizations/public';
-import { histogramVisTypeDefinition } from './histogram';
-import { lineVisTypeDefinition } from './line';
-import { areaVisTypeDefinition } from './area';
 import { heatmapVisTypeDefinition } from './heatmap';
-import { horizontalBarVisTypeDefinition } from './horizontal_bar';
 import { gaugeVisTypeDefinition } from './gauge';
 import { goalVisTypeDefinition } from './goal';
 
 export { pieVisTypeDefinition } from './pie';
 
 export const visLibVisTypeDefinitions: Array<VisTypeDefinition<any>> = [
-  histogramVisTypeDefinition,
-  lineVisTypeDefinition,
-  areaVisTypeDefinition,
-  heatmapVisTypeDefinition,
-  horizontalBarVisTypeDefinition,
-  gaugeVisTypeDefinition,
-  goalVisTypeDefinition,
-];
-
-export const convertedTypeDefinitions: Array<VisTypeDefinition<any>> = [
   heatmapVisTypeDefinition,
   gaugeVisTypeDefinition,
   goalVisTypeDefinition,

src/plugins/vis_types/vislib/public/vislib/VISLIB.md

@@ -1,4 +1,8 @@
-# Vislib general overview
+# Charts supported
+
+Vislib supports the heatmap and gauge/goal charts from the aggregation-based visualizations. It also contains the legacy implemementation of the pie chart (enabled by the visualization:visualize:legacyPieChartsLibrary advanced setting).
+
+# General overview
 
 `vis.js` constructor accepts vis parameters and render method accepts data. it exposes event emitter interface so we can listen to certain events like 'renderComplete'.
 
@@ -18,7 +22,4 @@ All base visualizations extend from `visualizations/_chart`
 
 ### Point series chart
 
-`visualizations/point_series` takes care of drawing the point series chart (no axes or titles, just the chart itself). It creates all the series defined and calls render method on them.
-
-currently there are 3 series types available (line, area, bars), they all extend from `visualizations/point_series/_point_series`.
-
+`visualizations/point_series` takes care of drawing the point series chart (no axes or titles, just the chart itself). It creates all the series defined and calls render method on them.

src/plugins/vis_types/vislib/public/vislib/_index.scss

@@ -5,5 +5,4 @@
 @import './components/tooltip/index';
 @import './components/legend/index';
 
-@import './visualizations/point_series/index';
 @import './visualizations/gauges/index';

src/plugins/vis_types/vislib/public/vislib/lib/axis/axis.test.js

@@ -96,7 +96,7 @@ describe('Vislib Axis Class Test Suite', function () {
 
     const visConfig = new VisConfig(
       {
-        type: 'histogram',
+        type: 'heatmap',
       },
       data,
       mockUiState,

src/plugins/vis_types/vislib/public/vislib/lib/axis/axis_title.test.js

@@ -103,7 +103,7 @@ describe('Vislib AxisTitle Class Test Suite', function () {
     dataObj = new Data(data, getMockUiState(), () => undefined);
     visConfig = new VisConfig(
       {
-        type: 'histogram',
+        type: 'heatmap',
       },
       data,
       getMockUiState(),

src/plugins/vis_types/vislib/public/vislib/lib/axis/x_axis.test.js

@@ -101,7 +101,7 @@ describe('Vislib xAxis Class Test Suite', function () {
 
     const visConfig = new VisConfig(
       {
-        type: 'histogram',
+        type: 'heatmap',
       },
       data,
       mockUiState,

src/plugins/vis_types/vislib/public/vislib/lib/axis/y_axis.test.js

@@ -81,7 +81,7 @@ function createData(seriesData) {
   buildYAxis = function (params) {
     const visConfig = new VisConfig(
       {
-        type: 'histogram',
+        type: 'heatmap',
       },
       data,
       mockUiState,

src/plugins/vis_types/vislib/public/vislib/lib/chart_title.test.js

@@ -99,7 +99,7 @@ describe('Vislib ChartTitle Class Test Suite', function () {
 
     const visConfig = new VisConfig(
       {
-        type: 'histogram',
+        type: 'heatmap',
         title: {
           text: 'rows',
         },