Sort palette nodes by component label (#2277)

Fixes #2171 Co-authored-by: Nick Bourdakos <[email protected]>
elyra-ai · Nov 5, 2021 · ef4d733 · ef4d733
1 parent 803e6d4
commit ef4d733
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/pipeline-editor/src/pipeline-hooks.spec.ts b/packages/pipeline-editor/src/pipeline-hooks.spec.ts
@@ -45,7 +45,8 @@ const createMockCategory = (
 
 const createMockComponent = (name: string): Component => {
   return {
-    op: name,
+    op: 'string',
+    label: name,
     id: 'string',
     type: 'execution_node',
     inputs: [],

diff --git a/packages/pipeline-editor/src/pipeline-hooks.ts b/packages/pipeline-editor/src/pipeline-hooks.ts
@@ -85,6 +85,7 @@ export interface IRuntimeComponent {
   node_types: {
     op: string;
     id: string;
+    label: string;
     type: 'execution_node';
     inputs: { app_data: any }[];
     outputs: { app_data: any }[];
@@ -138,7 +139,9 @@ export const sortPalette = (palette: {
 
   for (const components of palette.categories) {
     components.node_types.sort((a, b) =>
-      a.op.localeCompare(b.op, undefined, { numeric: true })
+      a.label.localeCompare(b.label, undefined, {
+        numeric: true
+      })
     );
   }
 };