[ColorMapping] Fix palette key mapping for legacy palette ids (#211858)

Currently, palettes with the legacy palette id of `eui_amsterdam_color_blind`, the old default id, gets assigned the `kibana 7` palette. The desired behavior is that this palette is assigned the new borealis default palette in `9.0`.
elastic · Feb 20, 2025 · 9e7d931 · 9e7d931
1 parent d3ef4c7
commit 9e7d931
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/platform/packages/shared/kbn-palettes/classes/palettes.ts b/src/platform/packages/shared/kbn-palettes/classes/palettes.ts
@@ -37,7 +37,7 @@ export class KbnPalettes {
 function buildAliasMappings(palettes: IKbnPalette[]): Map<string, string> {
   return palettes.reduce((acc, { id, aliases }) => {
     aliases.forEach((alias) => {
-      acc.set(alias, id);
+      if (!acc.has(alias)) acc.set(alias, id);
     });
     return acc;
   }, new Map());

diff --git a/src/platform/packages/shared/kbn-palettes/palettes/categorical/elastic.ts b/src/platform/packages/shared/kbn-palettes/palettes/categorical/elastic.ts
@@ -20,6 +20,7 @@ export const elasticPalette = new KbnColorFnPalette({
   id: KbnPalette.Default,
   type: 'categorical',
   aliases: [
+    KbnPalette.Default, // needed when switching between new and old themes
     'elastic_borealis', // placeholder - not yet used
     KbnPalette.Amsterdam, // to assign to existing default palettes
   ],