From 33431a5edc5a0a4c97abb95632d3c2c27cbcf6dd Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Mon, 2 Aug 2021 15:50:59 +0100
Subject: [PATCH] Rename visual actions from 'peek' to just 'open' screen in
 modal

---
 ...{CloseScreenPeekModal.svelte => CloseScreenModal.svelte} | 0
 .../PropertyControls/EventsEditor/actions/NavigateTo.svelte | 2 +-
 .../PropertyControls/EventsEditor/actions/index.js          | 6 +++---
 packages/client/src/components/PeekScreenDisplay.svelte     | 4 ++--
 packages/client/src/utils/buttonActions.js                  | 6 +++---
 5 files changed, 9 insertions(+), 9 deletions(-)
 rename packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/{CloseScreenPeekModal.svelte => CloseScreenModal.svelte} (100%)

diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/CloseScreenPeekModal.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/CloseScreenModal.svelte
similarity index 100%
rename from packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/CloseScreenPeekModal.svelte
rename to packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/CloseScreenModal.svelte
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/NavigateTo.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/NavigateTo.svelte
index 931c807ec9..ba4ccdb212 100644
--- a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/NavigateTo.svelte
+++ b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/NavigateTo.svelte
@@ -19,7 +19,7 @@
     {bindings}
   />
   <div />
-  <Checkbox text="Peek screen in modal" bind:value={parameters.peek} />
+  <Checkbox text="Open screen in modal" bind:value={parameters.peek} />
 </div>
 
 <style>
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/index.js b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/index.js
index 262f9362e4..c769a2300f 100644
--- a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/index.js
+++ b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/index.js
@@ -6,7 +6,7 @@ import TriggerAutomation from "./TriggerAutomation.svelte"
 import ValidateForm from "./ValidateForm.svelte"
 import LogOut from "./LogOut.svelte"
 import ClearForm from "./ClearForm.svelte"
-import CloseScreenPeekModal from "./CloseScreenPeekModal.svelte"
+import CloseScreenModal from "./CloseScreenModal.svelte"
 
 // Defines which actions are available to configure in the front end.
 // Unfortunately the "name" property is used as the identifier so please don't
@@ -49,7 +49,7 @@ export default [
     component: ClearForm,
   },
   {
-    name: "Close Screen-Peek Modal",
-    component: CloseScreenPeekModal,
+    name: "Close Screen Modal",
+    component: CloseScreenModal,
   },
 ]
diff --git a/packages/client/src/components/PeekScreenDisplay.svelte b/packages/client/src/components/PeekScreenDisplay.svelte
index 7d144d270e..2d4a08d2b5 100644
--- a/packages/client/src/components/PeekScreenDisplay.svelte
+++ b/packages/client/src/components/PeekScreenDisplay.svelte
@@ -24,7 +24,7 @@
     )
     // Listen for a close event to close the screen peek
     iframe.contentWindow.addEventListener(
-      "close-screen-peek",
+      "close-screen-modal",
       peekStore.actions.hidePeek
     )
     // Proxy notifications back to the parent window instead of iframe
@@ -38,7 +38,7 @@
       invalidateDataSource
     )
     iframe.contentWindow.removeEventListener(
-      "close-screen-peek",
+      "close-screen-modal",
       peekStore.actions.hidePeek
     )
     iframe.contentWindow.removeEventListener("notification", proxyNotification)
diff --git a/packages/client/src/utils/buttonActions.js b/packages/client/src/utils/buttonActions.js
index 03ed5ce2d0..408b35a705 100644
--- a/packages/client/src/utils/buttonActions.js
+++ b/packages/client/src/utils/buttonActions.js
@@ -99,10 +99,10 @@ const clearFormHandler = async (action, context) => {
   )
 }
 
-const closeScreenPeekModalHandler = () => {
+const closeScreenModalHandler = () => {
   // Emit this as a window event, so parent screens which are iframing us in
   // can close the modal
-  window.dispatchEvent(new Event("close-screen-peek"))
+  window.dispatchEvent(new Event("close-screen-modal"))
 }
 
 const handlerMap = {
@@ -115,7 +115,7 @@ const handlerMap = {
   ["Refresh Datasource"]: refreshDatasourceHandler,
   ["Log Out"]: logoutHandler,
   ["Clear Form"]: clearFormHandler,
-  ["Close Screen-Peek Modal"]: closeScreenPeekModalHandler,
+  ["Close Screen Modal"]: closeScreenModalHandler,
 }
 
 const confirmTextMap = {