clean up page state upon form reset

...as opposed to upon form population with no paletteKey.
AprilSylph · Feb 28, 2022 · 56aaa5e · 56aaa5e
1 parent d13e1ec
commit 56aaa5e
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/options/ui.js b/src/options/ui.js
@@ -41,10 +41,6 @@ const populateForm = ({ paletteKey, paletteData }) => {
     paletteForm.dataset.editing = paletteKey;
     createdTime.textContent = getTimestamp(paletteKey);
     deleteButton.disabled = false;
-  } else {
-    delete paletteForm.dataset.editing;
-    createdTime.textContent = '';
-    deleteButton.disabled = true;
   }
 
   for (const [propertyName, propertyValue] of Object.entries(paletteData)) {
@@ -97,6 +93,14 @@ const deleteCurrentPalette = async () => {
   }
 };
 
+const onFormReset = () => {
+  delete paletteForm.dataset.editing;
+  createdTime.textContent = '';
+
+  disableSaveButton();
+  deleteButton.disabled = true;
+};
+
 const onFormSubmitted = async event => {
   event.preventDefault();
 
@@ -159,7 +163,7 @@ getBuiltInPaletteList.then(builtInPaletteList => {
   ));
 });
 
-paletteForm.addEventListener('reset', disableSaveButton);
+paletteForm.addEventListener('reset', onFormReset);
 paletteForm.addEventListener('submit', onFormSubmitted);
 paletteForm.addEventListener('input', updatePreview);
 paletteForm.reset();