From 3fd6bdb2973c205b4546f91d66f5eec1f3922547 Mon Sep 17 00:00:00 2001
From: huanfeng <fasd9090@gmail.com>
Date: Sat, 28 Dec 2024 13:18:05 -0500
Subject: [PATCH] Implement sorting logic for llm tags display

---
 components/search/HierarchicalMenuWidget.tsx | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/components/search/HierarchicalMenuWidget.tsx b/components/search/HierarchicalMenuWidget.tsx
index f8c6e6628..26b0ba1d0 100644
--- a/components/search/HierarchicalMenuWidget.tsx
+++ b/components/search/HierarchicalMenuWidget.tsx
@@ -76,10 +76,11 @@ export const connectMultiselectHierarchicalMenu: MultiselectHierarchicalMenuConn
             attribute: string,
             isParent: boolean
           ): MultiselectHierarchicalMenuItem[] => {
-            // Safely attempt to retrieve facet values, default to an empty array if unavailable
+            const sortByParameter = isParent ? ["name:asc"] : ["count:desc"]
+
             const facetValues =
               (results?.getFacetValues(attribute, {
-                sortBy: ["name:asc"]
+                sortBy: sortByParameter
               }) as SearchResults.FacetValue[]) || []
 
             // Mapping over facetValues with an additional safety check
@@ -113,13 +114,7 @@ export const connectMultiselectHierarchicalMenu: MultiselectHierarchicalMenuConn
               return resultsItem ? { ...levelItem, ...resultsItem } : levelItem
             })
 
-            return mergedItems.sort((a, b) => {
-              // if (isParent) {
-              //   return a.label.localeCompare(b.label) // Alphabetical sort for parent
-              // } else {
-              return b.count - a.count // Sort by count descending for child
-              // }
-            })
+            return mergedItems
           }
 
           // Register refinements and items for each attribute.