From e1b5cf9a9a149434aad8d10a254c333c38f5f63f Mon Sep 17 00:00:00 2001
From: Lumi Pakkanen <lumi.pakkanen@gmail.com>
Date: Thu, 13 Jun 2024 17:51:33 +0300
Subject: [PATCH] Make it possible to cancel "Approximate by ratios"

ref #683
---
 cypress/e2e/basic.cy.ts                                |  2 +-
 src/components/ModifyScale.vue                         |  2 +-
 .../modals/modification/ApproximateByRatios.vue        | 10 ++++++++--
 src/stores/approximate-by-ratios.ts                    |  5 ++++-
 4 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/cypress/e2e/basic.cy.ts b/cypress/e2e/basic.cy.ts
index c7016060..98e369c1 100644
--- a/cypress/e2e/basic.cy.ts
+++ b/cypress/e2e/basic.cy.ts
@@ -49,7 +49,7 @@ describe("Scale generation/modification", () => {
     for (let i = 0; i < 5; ++i) {
       cy.get("button").contains("Apply").click();
     }
-    cy.get("button").contains("Close").click();
+    cy.get("button").contains("Done").click();
     cy.get("#scale-data").should("contain.value", "8/7");
   });
 });
\ No newline at end of file
diff --git a/src/components/ModifyScale.vue b/src/components/ModifyScale.vue
index 1d76350e..feaba26d 100644
--- a/src/components/ModifyScale.vue
+++ b/src/components/ModifyScale.vue
@@ -141,7 +141,7 @@ function clickSubset() {
 }
 
 function clickApproximateByRatios() {
-  approx.initialize()
+  approx.initialize(scale.sourceText)
   showApproximateByRatiosModal.value = true
 }
 
diff --git a/src/components/modals/modification/ApproximateByRatios.vue b/src/components/modals/modification/ApproximateByRatios.vue
index 5f9cec86..901f31cc 100644
--- a/src/components/modals/modification/ApproximateByRatios.vue
+++ b/src/components/modals/modification/ApproximateByRatios.vue
@@ -121,10 +121,15 @@ function modify(expand = true) {
   scale.computeScale()
   emit('done')
 }
+
+function cancel() {
+  scale.sourceText = approx.originalSource
+  emit('done')
+}
 </script>
 
 <template>
-  <Modal @confirm="modify(true)" @cancel="modify(false)">
+  <Modal @confirm="modify(true)" @cancel="cancel">
     <template #header>
       <h2>Approximate by ratios</h2>
     </template>
@@ -233,8 +238,9 @@ function modify(expand = true) {
     <template #footer>
       <div class="btn-group">
         <button @click="modifyAndAdvance">Apply</button>
-        <button @click="modify(true)">Close</button>
+        <button @click="modify(true)">Done</button>
         <button @click="modify(false)">Raw</button>
+        <button @click="cancel">Cancel</button>
       </div>
     </template>
   </Modal>
diff --git a/src/stores/approximate-by-ratios.ts b/src/stores/approximate-by-ratios.ts
index 9b1550c8..4f3fbab5 100644
--- a/src/stores/approximate-by-ratios.ts
+++ b/src/stores/approximate-by-ratios.ts
@@ -12,6 +12,7 @@ export const useApproximateByRatiosStore = defineStore('approximate-by-ratios',
   const oddLimit = ref(9)
   const primeLimit = ref(7)
   const maxExponent = ref(2)
+  const originalSource = ref('')
 
   const safeOddLimit = computed(() => clamp(3, 101, 2 * Math.floor(oddLimit.value / 2) + 1))
   const safePrimeLimit = computed(() => {
@@ -92,9 +93,10 @@ export const useApproximateByRatiosStore = defineStore('approximate-by-ratios',
     return '>97-limit'
   }
 
-  function initialize() {
+  function initialize(source: string) {
     degree.value = 1
     approximationIndex.value = 0
+    originalSource.value = source
   }
 
   return {
@@ -110,6 +112,7 @@ export const useApproximateByRatiosStore = defineStore('approximate-by-ratios',
     safeOddLimit,
     safePrimeLimit,
     safeMaxExponent,
+    originalSource,
     modifyPrimeLimit,
     primeLimitString,
     initialize