[Vis colors] Update legacy mapped colors in charts plugin to use ouiP…

…aletteColorBlind()

Signed-off-by: Manasvini B Suryanarayana <manasvis@amazon.com>

CHANGELOG.md

@@ -64,6 +64,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - [Vis Colors] [Maps] Replace hardcoded color to OUI color in `maps_legacy` plugin ([#4294](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4294))
 - [Vis Colors] [TSVB] Update default color in `vis_type_timeseries` to use `ouiPaletteColorBlind()[0]`([#4363](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4363))
 - [Vis Colors] [Timeline] Replace `vis_type_timeline` colors with `ouiPaletteColorBlind()` ([#4366](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4366))
+- [Vis colors] Update legacy mapped colors in charts plugin to use `ouiPaletteColorBlind()`, Update default color in legacy visualizations to use `ouiPaletteColorBlind()[0]` ([#4398](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4398))
 
 ### 🔩 Tests
 

src/plugins/charts/public/services/colors/colors.test.ts

@@ -29,8 +29,8 @@
  */
 
 import { coreMock } from '../../../../../core/public/mocks';
+import { euiPaletteColorBlind } from '@elastic/eui';
 import { COLOR_MAPPING_SETTING } from '../../../common';
-import { seedColors } from './seed_colors';
 import { ColorsService } from './colors';
 
 // Local state for config
@@ -138,7 +138,7 @@ describe('Vislib Color Service', () => {
  });
 
  it('should return the first hex color in the seed colors array', () => {
- expect(color(arr[0])).toBe(seedColors[0]);
+ expect(color(arr[0])).toBe(euiPaletteColorBlind()[0]);
  });
 
  it('should return the value from the mapped colors', () => {

src/plugins/charts/public/services/colors/mapped_colors.ts

@@ -33,8 +33,8 @@ import Color from 'color';
 
 import { CoreSetup } from 'opensearch-dashboards/public';
 
+import { euiPaletteColorBlind } from '@elastic/eui';
 import { COLOR_MAPPING_SETTING } from '../../../common';
-import { createColorPalette } from './color_palette';
 
 const standardizeColor = (color: string) => new Color(color).hex().toLowerCase();
 
@@ -100,7 +100,11 @@ export class MappedColors {
 
  // Generate a color palette big enough that all new keys can have unique color values
  const allColors = _(this._mapping).values().union(configColors).union(oldColors).value();
- const colorPalette = createColorPalette(allColors.length + keysToMap.length);
+ const numColors = allColors.length + keysToMap.length;
+ const colorPalette = euiPaletteColorBlind({
+ rotations: Math.ceil(numColors / 10),
+ direction: 'both',
+ }).slice(0, numColors);
  let newColors = _.difference(colorPalette, allColors);
 
  while (keysToMap.length > newColors.length) {