From ff93d306080c8001e7a3dacb6c3a027e7eb8b963 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Wed, 6 Jan 2021 16:03:00 +0000
Subject: [PATCH 1/4] Add toggle to make autoscreens optional

---
 .../modals/CreateTableModal.svelte            | 42 +++++++++++--------
 1 file changed, 24 insertions(+), 18 deletions(-)

diff --git a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
index 7c26e236a6..e6883df1a4 100644
--- a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
+++ b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
@@ -1,8 +1,8 @@
 <script>
-  import { goto, params } from "@sveltech/routify"
+  import { goto } from "@sveltech/routify"
   import { backendUiStore, store } from "builderStore"
   import { notifier } from "builderStore/store/notifications"
-  import { Input, Label, ModalContent } from "@budibase/bbui"
+  import { Input, Label, ModalContent, Toggle } from "@budibase/bbui"
   import TableDataImport from "../TableDataImport.svelte"
   import analytics from "analytics"
   import screenTemplates from "builderStore/store/screenTemplates"
@@ -20,6 +20,7 @@
   let name
   let dataImport
   let error = ""
+  let createAutoscreens = true
 
   function checkValid(evt) {
     const tableName = evt.target.value
@@ -48,23 +49,25 @@
     analytics.captureEvent("Table Created", { name })
 
     // Create auto screens
-    const screens = screenTemplates($store, [table])
-      .filter(template => defaultScreens.includes(template.id))
-      .map(template => template.create())
-    for (let screen of screens) {
-      // Record the table that created this screen so we can link it later
-      screen.autoTableId = table._id
-      await store.actions.screens.create(screen)
-    }
+    if (createAutoscreens) {
+      const screens = screenTemplates($store, [table])
+        .filter(template => defaultScreens.includes(template.id))
+        .map(template => template.create())
+      for (let screen of screens) {
+        // Record the table that created this screen so we can link it later
+        screen.autoTableId = table._id
+        await store.actions.screens.create(screen)
+      }
 
-    // Create autolink to newly created list screen
-    const listScreen = screens.find(screen =>
-      screen.props._instanceName.endsWith("List")
-    )
-    await store.actions.components.links.save(
-      listScreen.routing.route,
-      table.name
-    )
+      // Create autolink to newly created list screen
+      const listScreen = screens.find(screen =>
+        screen.props._instanceName.endsWith("List")
+      )
+      await store.actions.components.links.save(
+        listScreen.routing.route,
+        table.name
+      )
+    }
 
     // Navigate to new table
     $goto(`./table/${table._id}`)
@@ -83,6 +86,9 @@
     on:input={checkValid}
     bind:value={name}
     {error} />
+  <Toggle
+    text="Create screens to view and edit items in this table"
+    bind:checked={createAutoscreens} />
   <div>
     <Label grey extraSmall>Create Table from CSV (Optional)</Label>
     <TableDataImport bind:dataImport />

From fcad9892e04a841f6942dcceac9acc590e5b0cf5 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Wed, 6 Jan 2021 16:03:45 +0000
Subject: [PATCH 2/4] Swap back to border for selected component highlighting
 from box-shadow

---
 packages/client/src/utils/styleable.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/client/src/utils/styleable.js b/packages/client/src/utils/styleable.js
index a3c4cb7ca5..3e934016f6 100644
--- a/packages/client/src/utils/styleable.js
+++ b/packages/client/src/utils/styleable.js
@@ -36,7 +36,7 @@ const addBuilderPreviewStyles = (styleString, componentId, selectable) => {
 
     // Highlighted selected element
     if (componentId === state.selectedComponentId) {
-      str += `;box-shadow: 0 0 0 ${selectedComponentWidth}px ${selectedComponentColor} inset !important;`
+      str += `;border: ${selectedComponentWidth}px solid ${selectedComponentColor} !important;`
     }
   }
 

From a5b574d61ecf763c93d459ac6375aa3466032547 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Thu, 7 Jan 2021 09:32:44 +0000
Subject: [PATCH 3/4] Small text change to autoscreen toggle

---
 .../backend/TableNavigator/modals/CreateTableModal.svelte       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
index e6883df1a4..819af1f27b 100644
--- a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
+++ b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
@@ -87,7 +87,7 @@
     bind:value={name}
     {error} />
   <Toggle
-    text="Create screens to view and edit items in this table"
+    text="Create screens to view and edit rows in this table"
     bind:checked={createAutoscreens} />
   <div>
     <Label grey extraSmall>Create Table from CSV (Optional)</Label>

From c5263999c31ec73a9edb72bb4e85258733d31914 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Thu, 7 Jan 2021 10:00:24 +0000
Subject: [PATCH 4/4] Update autoscreen toggle text

---
 .../backend/TableNavigator/modals/CreateTableModal.svelte       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
index 819af1f27b..3338072ec2 100644
--- a/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
+++ b/packages/builder/src/components/backend/TableNavigator/modals/CreateTableModal.svelte
@@ -87,7 +87,7 @@
     bind:value={name}
     {error} />
   <Toggle
-    text="Create screens to view and edit rows in this table"
+    text="Generate screens in the design section"
     bind:checked={createAutoscreens} />
   <div>
     <Label grey extraSmall>Create Table from CSV (Optional)</Label>