pipelineList: expand group when only 1 exists

AdamNowotny · Sep 6, 2024 · 993b749 · 993b749
1 parent 77a835d
commit 993b749
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/src/options/pages/service/components/pipelineList/pipelineList.tsx b/src/options/pages/service/components/pipelineList/pipelineList.tsx
@@ -45,17 +45,21 @@ export default ({
     const groupNames: string[] = Array.from(groups.keys());
     return (
         <div>
-            {groupNames.map((key: string) => (
-                <GroupPanel
-                    key={key}
-                    name={key}
-                    items={groups.get(key) ?? []}
-                    selectedItems={selectedItems}
-                    filter={filter}
-                    onChanged={handleChanged}
-                    onAllChanged={handleAllChanged}
-                />
-            ))}
+            {groupNames.map((key: string) => {
+                const groupItems = groups.get(key) ?? [];
+                return (
+                    <GroupPanel
+                        key={key}
+                        name={key}
+                        items={groupItems}
+                        selectedItems={selectedItems}
+                        filter={filter}
+                        defaultOpen={groupNames.length === 1}
+                        onChanged={handleChanged}
+                        onAllChanged={handleAllChanged}
+                    />
+                );
+            })}
         </div>
     );
 };
@@ -65,13 +69,15 @@ const GroupPanel = ({
     items,
     filter,
     selectedItems = [],
+    defaultOpen,
     onChanged,
     onAllChanged,
 }: {
     name: string;
     items: CIPipeline[];
     filter?: string;
     selectedItems?: string[];
+    defaultOpen?: boolean;
     onChanged: (id: string, checked: boolean) => void;
     onAllChanged: (ids: string[], checked: boolean) => void;
 }) => {
@@ -86,7 +92,7 @@ const GroupPanel = ({
     const someVisibleChecked = filteredItems.some(item =>
         selectedItems.includes(item.id),
     );
-    const [open, setOpen] = useState(someVisibleChecked);
+    const [open, setOpen] = useState(defaultOpen ?? someVisibleChecked);
     if (!open && filter) setOpen(true);
 
     const checkAll = e => {