style: Use correct combo chart icons

visualize-admin · Oct 17, 2023 · 9f2b0a7 · 9f2b0a7
1 parent aa1547c
commit 9f2b0a7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/app/configurator/components/ui-helpers.ts b/app/configurator/components/ui-helpers.ts
@@ -165,10 +165,11 @@ export const getIconName = (name: string): IconName => {
     case "map":
       return "chartMap";
     case "comboLineSingle":
+      return "chartMultiLine";
     case "comboLineDual":
+      return "chartDualAxisLine";
     case "comboLineColumn":
-      // FIXME: Use a proper icon.
-      return "star";
+      return "chartColumnLine";
     case "baseLayer":
       return "mapMaptype";
     case "areaLayer":

diff --git a/app/icons/index.tsx b/app/icons/index.tsx
@@ -51,11 +51,12 @@ export const getChartIcon = (chartType: ChartType): IconName => {
       return "chartScatterplot";
     case "table":
       return "table";
-    // FIXME: use proper icons
     case "comboLineSingle":
+      return "chartMultiLine";
     case "comboLineDual":
+      return "chartDualAxisLine";
     case "comboLineColumn":
-      return "star";
+      return "chartColumnLine";
     default:
       const _exhaustiveCheck: never = chartType;
       return _exhaustiveCheck;