Dismiss modal on save

MuckRock · Jan 8, 2024 · f70c95d · f70c95d
1 parent f838b9c
commit f70c95d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/src/common/dialog/RevisionsDialog.svelte b/src/common/dialog/RevisionsDialog.svelte
@@ -89,7 +89,11 @@
       {/if}
 
       {#if isPremiumOrg(user?.organization)}
-        <form>
+        <form
+          on:submit={(event) => {
+            event.preventDefault();
+          }}
+        >
           <label class="revision-control-input">
             <input
               type="checkbox"
@@ -99,11 +103,7 @@
             {$_("dialogRevisionsDialog.controlLabel")}
           </label>
           <div class="buttons">
-            <Button
-              on:click={(event) => {
-                event.preventDefault();
-                onSave(enabled);
-              }}>{$_("dialog.save")}</Button
+            <Button on:click={() => onSave(enabled)}>{$_("dialog.save")}</Button
             >
             <Button secondary={true} on:click={onCancel}>
               {$_("dialog.cancel")}

diff --git a/src/common/dialog/RevisionsDialogContainer.svelte b/src/common/dialog/RevisionsDialogContainer.svelte
@@ -12,9 +12,13 @@
   const documentId = document?.id;
   const revisions = document?.revisions;
   const enabled = document?.revisionControl;
-  const onSave = (enabled: boolean) =>
+  const onSave = (enabled: boolean) => {
     changeRevisionControl([documentId], enabled);
-  const onCancel = () => emit.dismiss();
+    emit.dismiss();
+  };
+  const onCancel = () => {
+    emit.dismiss();
+  };
 </script>
 
 <RevisionsDialog {documentId} {revisions} {enabled} {onSave} {onCancel} />