From c2cc99ef2c3e92e5c2610ad958e1947149b9b69b Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Wed, 12 May 2021 14:34:25 +0100
Subject: [PATCH] Fix chart placeholder, fix spectrum colours and begin work on
 pagination component

---
 packages/bbui/src/Table/Table.svelte          |  8 ++----
 .../DataProviderSelect.svelte                 | 16 +++++++++++-
 packages/builder/src/global.css               |  4 +--
 packages/standard-components/manifest.json    | 13 ++++++++++
 .../standard-components/src/Pagination.svelte | 25 +++++++++++++++++++
 .../src/charts/ApexChart.svelte               | 12 ++++++---
 6 files changed, 66 insertions(+), 12 deletions(-)
 create mode 100644 packages/standard-components/src/Pagination.svelte

diff --git a/packages/bbui/src/Table/Table.svelte b/packages/bbui/src/Table/Table.svelte
index 048ded2b5b..56f1ac96a8 100644
--- a/packages/bbui/src/Table/Table.svelte
+++ b/packages/bbui/src/Table/Table.svelte
@@ -345,7 +345,7 @@
 
 <style>
   .wrapper {
-    background-color: var(--spectrum-alias-background-color-secondary);
+    background-color: var(--spectrum-alias-background-color-default);
     overflow: hidden;
     position: relative;
     z-index: 1;
@@ -389,7 +389,7 @@
     position: sticky;
     top: 0;
     z-index: 2;
-    background-color: var(--spectrum-alias-background-color-secondary);
+    background-color: var(--spectrum-alias-background-color-default);
     border-bottom: 1px solid
       var(--spectrum-table-border-color, var(--spectrum-alias-border-color-mid));
   }
@@ -462,10 +462,6 @@
   tbody tr.hidden {
     height: calc(var(--row-height) + 1px);
   }
-  tbody tr.offset {
-    background-color: red;
-    display: block;
-  }
   td {
     padding-top: 0;
     padding-bottom: 0;
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/DataProviderSelect.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/DataProviderSelect.svelte
index 9be8feab57..f21cc7fcf9 100644
--- a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/DataProviderSelect.svelte
+++ b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/DataProviderSelect.svelte
@@ -3,20 +3,34 @@
   import { makePropSafe } from "@budibase/string-templates"
   import { currentAsset, store } from "builderStore"
   import { findComponentPath } from "builderStore/storeUtils"
+  import { createEventDispatcher, onMount } from "svelte"
 
   export let value
+  export let onChange
+
+  const dispatch = createEventDispatcher()
+  const getValue = component => `{{ literal ${makePropSafe(component._id)} }}`
 
   $: path = findComponentPath($currentAsset.props, $store.selectedComponentId)
   $: providers = path.filter(
     component =>
       component._component === "@budibase/standard-components/dataprovider"
   )
+
+  // Set initial value to closest data provider
+  onMount(() => {
+    const valid = value && providers.find(x => getValue(x) === value) != null
+    if (!valid && providers.length) {
+      dispatch("change", getValue(providers[providers.length - 1]))
+    }
+  })
 </script>
 
 <Select
   {value}
+  placeholder={null}
   on:change
   options={providers}
   getOptionLabel={component => component._instanceName}
-  getOptionValue={component => `{{ literal ${makePropSafe(component._id)} }}`}
+  getOptionValue={getValue}
 />
diff --git a/packages/builder/src/global.css b/packages/builder/src/global.css
index 726f422a48..09c8dd2a34 100644
--- a/packages/builder/src/global.css
+++ b/packages/builder/src/global.css
@@ -7,12 +7,12 @@ html, body {
 }
 
 .spectrum--light {
-    --spectrum-alias-background-color-primary: var(--spectrum-global-color-gray-75);
+    /*--spectrum-alias-background-color-primary: var(--spectrum-global-color-gray-75);*/
 }
 
 body {
     --background: var(--spectrum-alias-background-color-primary);
-    --background-alt: var(--spectrum-alias-background-color-secondary);
+    --background-alt: var(--spectrum-alias-background-color-default);
     --border-light: 1px solid var(--spectrum-global-color-gray-300);
     --border-dark: 1px solid var(--spectrum-global-color-gray-400);
     --ink: var(--spectrum-alias-text-color);
diff --git a/packages/standard-components/manifest.json b/packages/standard-components/manifest.json
index c20744a1a2..6a6b00732b 100644
--- a/packages/standard-components/manifest.json
+++ b/packages/standard-components/manifest.json
@@ -1558,5 +1558,18 @@
     "context": {
       "type": "schema"
     }
+  },
+  "pagination": {
+    "name": "Pagination Control",
+    "icon": "",
+    "styleable": true,
+    "hasChildren": "false",
+    "settings": [
+      {
+        "type": "dataProvider",
+        "label": "Provider",
+        "key": "dataProviderId"
+      }
+    ]
   }
 }
diff --git a/packages/standard-components/src/Pagination.svelte b/packages/standard-components/src/Pagination.svelte
new file mode 100644
index 0000000000..12919cba8c
--- /dev/null
+++ b/packages/standard-components/src/Pagination.svelte
@@ -0,0 +1,25 @@
+<script>
+  import { getContext } from "svelte"
+
+  export let dataProviderId
+
+  const component = getContext("component")
+  const { styleable, ActionTypes, builderStore } = getContext("sdk")
+  const dataContext = getContext("context")
+
+  $: dataProviderContext = dataContext?.[dataProviderId]
+  $: pageNumber = dataProviderContext?.pageNumber ?? 1
+  $: hasPrevPage = dataProviderContext?.hasPrevPage ?? false
+  $: hasNextPage = dataProviderContext?.hasNextPage ?? false
+  $: prevPage = dataContext?.[`${dataProviderId}_${ActionTypes.PrevPage}`]
+  $: nextPage = dataContext?.[`${dataProviderId}_${ActionTypes.NextPage}`]
+  $: hasValidContext = dataProviderContext != null && nextPage != null
+</script>
+
+{#if hasValidContext}
+  <div>
+    Page {pageNumber}
+  </div>
+{:else if builderStore.inBuilder}
+  <div>Choose a data provider to control with this pagination component.</div>
+{/if}
diff --git a/packages/standard-components/src/charts/ApexChart.svelte b/packages/standard-components/src/charts/ApexChart.svelte
index cf9cade436..a7e25514e0 100644
--- a/packages/standard-components/src/charts/ApexChart.svelte
+++ b/packages/standard-components/src/charts/ApexChart.svelte
@@ -10,9 +10,9 @@
 
 {#if options}
   <div use:chart={options} use:styleable={$component.styles} />
-{:else if builderStore.inBuilder}
-  <div use:styleable={$component.styles}>
-    Use the settings panel to build your chart -->
+{:else if $builderStore.inBuilder}
+  <div class="placeholder" use:styleable={$component.styles}>
+    Use the settings panel to build your chart.
   </div>
 {/if}
 
@@ -21,4 +21,10 @@
     display: flex !important;
     text-transform: capitalize;
   }
+  div :global(.apexcharts-yaxis-label, .apexcharts-xaxis-label) {
+    fill: #aaa;
+  }
+  div.placeholder {
+    padding: 10px;
+  }
 </style>