Minor adjustments to PR#2137

hackforla · Feb 25, 2025 · 4e1f678 · 4e1f678
1 parent 6b198d8
commit 4e1f678
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 13 deletions.
diff --git a/client/src/components/Modals/ActionProjectCopy.jsx b/client/src/components/Modals/ActionProjectCopy.jsx
@@ -58,7 +58,7 @@ export default function CopyProjectModal({
         <div style={theme.typography.heading3}>{selectedProjectName}</div>
         <div style={{ margin: "1.5rem 2.5rem 1.5rem 0.75rem" }}>
           <input
-            placeholder="Name of Duplicated Project"
+            placeholder="Name of the duplicated project"
             type="text"
             id="duplicateName"
             name="duplicateName"
@@ -75,6 +75,7 @@ export default function CopyProjectModal({
           <Button
             onClick={() => onClose("ok", duplicateProjectName)}
             variant="primary"
+            disabled={!duplicateProjectName}
           >
             Create a Copy
           </Button>

diff --git a/client/src/components/Modals/ActionProjectSnapshot.jsx b/client/src/components/Modals/ActionProjectSnapshot.jsx
@@ -35,8 +35,7 @@ export default function SnapshotProjectModal({
 }) {
   const theme = useTheme();
   const classes = useStyles({ theme });
-  const [snapshotProjectName, setSnapshotProjectName] =
-    useState(selectedProjectName);
+  const [snapshotProjectName, setSnapshotProjectName] = useState("");
 
   return (
     <ModalDialog
@@ -55,7 +54,7 @@ export default function SnapshotProjectModal({
         </div>
         <div className={classes.subheading} style={{ width: "75%" }}>
           <input
-            placeholder="Name of Snapshot Project"
+            placeholder="Name of the Snapshot"
             type="text"
             id="duplicateName"
             name="duplicateName"
@@ -64,16 +63,13 @@ export default function SnapshotProjectModal({
           />
         </div>
         <div className={classes.buttonFlexBox}>
-          <Button
-            className={classes.buttonColor}
-            onClick={onClose}
-            variant="secondary"
-          >
+          <Button variant="secondary" onClick={onClose}>
             Cancel
           </Button>
           <Button
             onClick={() => onClose("ok", snapshotProjectName)}
             variant="primary"
+            disabled={!snapshotProjectName}
           >
             Done
           </Button>

diff --git a/client/src/components/Modals/ActionSnapshotRename.jsx b/client/src/components/Modals/ActionSnapshotRename.jsx
@@ -39,8 +39,7 @@ export default function RenameSnapshotModal({
 }) {
   const theme = useTheme();
   const classes = useStyles({ theme });
-  const [snapshotProjectName, setSnapshotProjectName] =
-    useState(selectedProjectName);
+  const [snapshotProjectName, setSnapshotProjectName] = useState("");
 
   return (
     <ModalDialog
@@ -51,14 +50,14 @@ export default function RenameSnapshotModal({
       <div className={classes.container}>
         <div className={classes.heading1}>
           <MdCameraAlt className={classes.icon} />
-          Rename Snapshot
+          Rename Snapshot &quot;{selectedProjectName}&quot;
         </div>
         <div className={classes.subheading}>
           Enter a new name for your snapshot
         </div>
         <div className={classes.subheading} style={{ width: "75%" }}>
           <input
-            placeholder="Name of Duplicated Project"
+            placeholder="New project name for snapshot"
             type="text"
             id="duplicateName"
             name="duplicateName"
@@ -77,6 +76,7 @@ export default function RenameSnapshotModal({
           <Button
             onClick={() => onClose("ok", snapshotProjectName)}
             variant="primary"
+            disabled={!snapshotProjectName}
           >
             Done
           </Button>