feat: Feed Selection optimizations

Author: PintoGideon

src/components/Feeds/FeedListView.tsx

@@ -1,4 +1,4 @@
-import type { Feed } from "@fnndsc/chrisapi";
+import type { Feed, FileBrowserFolder } from "@fnndsc/chrisapi";
 import { ChartDonutUtilization } from "@patternfly/react-charts";
 import {
   Button,
@@ -384,17 +384,27 @@ const TableRow: React.FC<TableRowProps> = ({
   const navigate = useNavigate();
   const { isDarkTheme } = useContext(ThemeContext);
 
+  // Add a cache for folder data with proper typing
+  const folderCache = useRef<FileBrowserFolder | null>(null);
+
   /**
    * Track row progress state for background color
    */
   const [rowProgress, setRowProgress] = useState<number | null>(null);
   const [rowError, setRowError] = useState<boolean>(false);
 
   /**
-   * Get folder for this feed
+   * Get folder for this feed - with caching
    */
   const getFolderForThisFeed = async () => {
+    // Return cached data if available
+    if (folderCache.current) {
+      return folderCache.current;
+    }
+
+    // Otherwise fetch and cache
     const payload = await feed.getFolder();
+    folderCache.current = payload;
     return payload;
   };
 
@@ -455,7 +465,22 @@ const TableRow: React.FC<TableRowProps> = ({
             onSelect: async (event) => {
               event.stopPropagation();
               const isChecked = event.currentTarget.checked;
-              const payload = await getFolderForThisFeed();
+
+              // Only fetch folder data if the checkbox is being checked
+              // This prevents the delay when simply rendering checkboxes
+              let payload: FileBrowserFolder | null = null;
+              if (isChecked) {
+                payload = await getFolderForThisFeed();
+              } else if (isSelected) {
+                // If unchecking, we don't need to fetch again, just use the path
+                handlers.handleCheckboxChange(
+                  event,
+                  feed.data.folder_path,
+                  null,
+                  "folder",
+                );
+                return;
+              }
 
               /**
                * Create a new event object with the captured value

src/components/NewLibrary/utils/longpress.tsx

@@ -118,14 +118,18 @@ export default function useLongPress() {
     e: React.FormEvent<HTMLInputElement>,
     path: string,
     payload:
-      | FileBrowserFolder
-      | FileBrowserFolderFile
-      | FileBrowserFolderLinkFile,
+      | (FileBrowserFolder | FileBrowserFolderFile | FileBrowserFolderLinkFile)
+      | null, // null is valid for deselection operations
     type: string,
   ) => {
     if (e.currentTarget.checked) {
-      selectFolder(path, type, payload);
+      // For selection, we need a valid payload
+      if (payload) {
+        selectFolder(path, type, payload);
+      }
     } else {
+      // For deselection, we only need the path
+      // The payload can be null in this case
       deselectFolder(path);
     }
   };