From 6f064813eaa0b87f282a7fceb317f9c374db7a82 Mon Sep 17 00:00:00 2001
From: amhsirak <karishmashuklaa@gmail.com>
Date: Mon, 4 Nov 2024 21:10:02 +0530
Subject: [PATCH 1/2] feat: disable capture list initial stage confirmation if
 !listSelector and browser steps

---
 src/components/organisms/RightSidePanel.tsx | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/src/components/organisms/RightSidePanel.tsx b/src/components/organisms/RightSidePanel.tsx
index f2871aea..f578b485 100644
--- a/src/components/organisms/RightSidePanel.tsx
+++ b/src/components/organisms/RightSidePanel.tsx
@@ -1,4 +1,4 @@
-import React, { useState, useCallback, useEffect } from 'react';
+import React, { useState, useCallback, useEffect, useMemo } from 'react';
 import { Button, Paper, Box, TextField, IconButton } from "@mui/material";
 import EditIcon from '@mui/icons-material/Edit';
 import TextFieldsIcon from '@mui/icons-material/TextFields';
@@ -373,6 +373,19 @@ export const RightSidePanel: React.FC<RightSidePanelProps> = ({ onFinishCapture
     stopGetScreenshot();
   };
 
+  const isConfirmCaptureDisabled = useMemo(() => {
+    // Check if we are in the initial stage and if there are no browser steps or no valid list selectors with fields
+    if (captureStage !== 'initial') return false;
+  
+    const hasValidListSelector = browserSteps.some(step => 
+      step.type === 'list' && 
+      step.listSelector && 
+      Object.keys(step.fields).length > 0
+    );
+  
+    return !hasValidListSelector;
+  }, [captureStage, browserSteps]);
+
   return (
     <Paper sx={{ height: '520px', width: 'auto', alignItems: "center", background: 'inherit' }} id="browser-actions" elevation={0}>
       {/* <SimpleBox height={60} width='100%' background='lightGray' radius='0%'>
@@ -387,7 +400,7 @@ export const RightSidePanel: React.FC<RightSidePanelProps> = ({ onFinishCapture
               <Button
                 variant="outlined"
                 onClick={handleConfirmListCapture}
-                disabled={hasUnconfirmedListTextFields}
+                disabled={captureStage === 'initial' ? isConfirmCaptureDisabled : hasUnconfirmedListTextFields}
               >
                 {captureStage === 'initial' ? 'Confirm Capture' :
                   captureStage === 'pagination' ? 'Confirm Pagination' :

From 81e869829155e67fa11fd5b36175c1439581673f Mon Sep 17 00:00:00 2001
From: amhsirak <karishmashuklaa@gmail.com>
Date: Mon, 4 Nov 2024 21:20:58 +0530
Subject: [PATCH 2/2] feat: add check for unconfirmed list text fields

---
 src/components/organisms/RightSidePanel.tsx | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/components/organisms/RightSidePanel.tsx b/src/components/organisms/RightSidePanel.tsx
index f578b485..a11989bd 100644
--- a/src/components/organisms/RightSidePanel.tsx
+++ b/src/components/organisms/RightSidePanel.tsx
@@ -383,8 +383,9 @@ export const RightSidePanel: React.FC<RightSidePanelProps> = ({ onFinishCapture
       Object.keys(step.fields).length > 0
     );
   
-    return !hasValidListSelector;
-  }, [captureStage, browserSteps]);
+     // Disable the button if there are no valid list selectors or if there are unconfirmed list text fields
+    return !hasValidListSelector || hasUnconfirmedListTextFields;
+  }, [captureStage, browserSteps, hasUnconfirmedListTextFields]);
 
   return (
     <Paper sx={{ height: '520px', width: 'auto', alignItems: "center", background: 'inherit' }} id="browser-actions" elevation={0}>