From bd4a804a969b4217120bd39cdeb85ceeeea7adb0 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <andrew@kingston.dev>
Date: Thu, 22 Aug 2024 12:07:12 +0100
Subject: [PATCH] Add accent colors to all buttons

---
 .../bbui/src/ActionButton/ActionButton.svelte | 22 +++++++++++++++++++
 packages/bbui/src/helpers.js                  | 10 +++++++++
 .../buttons/TableFilterButton.svelte          |  9 +++++++-
 .../buttons/grid/GridAutomationsButton.svelte |  9 ++++++--
 .../grid/GridColumnsSettingButton.svelte      |  1 +
 .../buttons/grid/GridRowActionsButton.svelte  | 13 +++++++----
 .../buttons/grid/GridScreensButton.svelte     |  9 +++++++-
 7 files changed, 65 insertions(+), 8 deletions(-)

diff --git a/packages/bbui/src/ActionButton/ActionButton.svelte b/packages/bbui/src/ActionButton/ActionButton.svelte
index 7019ad7d2c..8b2d11232c 100644
--- a/packages/bbui/src/ActionButton/ActionButton.svelte
+++ b/packages/bbui/src/ActionButton/ActionButton.svelte
@@ -3,6 +3,7 @@
   import { createEventDispatcher } from "svelte"
   import Tooltip from "../Tooltip/Tooltip.svelte"
   import { fade } from "svelte/transition"
+  import { hexToRGBA } from "../helpers"
 
   const dispatch = createEventDispatcher()
 
@@ -17,9 +18,22 @@
   export let fullWidth = false
   export let noPadding = false
   export let tooltip = ""
+  export let accentColor = null
 
   let showTooltip = false
 
+  $: accentStyle = getAccentStyle(accentColor)
+
+  const getAccentStyle = color => {
+    if (!color) {
+      return ""
+    }
+    let style = ""
+    style += `--accent-bg-color:${hexToRGBA(color, 0.15)};`
+    style += `--accent-border-color:${hexToRGBA(color, 0.35)};`
+    return style
+  }
+
   function longPress(element) {
     if (!longPressable) return
     let timer
@@ -47,6 +61,7 @@
   on:mouseover={() => (showTooltip = true)}
   on:mouseleave={() => (showTooltip = false)}
   on:focus={() => (showTooltip = true)}
+  style={accentStyle}
 >
   <button
     use:longPress
@@ -58,6 +73,7 @@
     class="spectrum-ActionButton spectrum-ActionButton--size{size}"
     class:active
     class:disabled
+    class:accent={accentColor != null}
     {disabled}
     on:longPress
     on:click|preventDefault
@@ -138,4 +154,10 @@
     text-align: center;
     z-index: 1;
   }
+
+  button.accent.is-selected,
+  button:active.accent {
+    border: 1px solid var(--accent-border-color);
+    background: var(--accent-bg-color);
+  }
 </style>
diff --git a/packages/bbui/src/helpers.js b/packages/bbui/src/helpers.js
index 0f912a7161..246587af44 100644
--- a/packages/bbui/src/helpers.js
+++ b/packages/bbui/src/helpers.js
@@ -228,3 +228,13 @@ export const getDateDisplayValue = (
     return value.format(`${localeDateFormat} HH:mm`)
   }
 }
+
+export const hexToRGBA = (color, opacity) => {
+  if (color.includes("#")) {
+    color = color.replace("#", "")
+  }
+  const r = parseInt(color.substring(0, 2), 16)
+  const g = parseInt(color.substring(2, 4), 16)
+  const b = parseInt(color.substring(4, 6), 16)
+  return `rgba(${r}, ${g}, ${b}, ${opacity})`
+}
diff --git a/packages/builder/src/components/backend/DataTable/buttons/TableFilterButton.svelte b/packages/builder/src/components/backend/DataTable/buttons/TableFilterButton.svelte
index 91ec9a9b01..c4bdc653c3 100644
--- a/packages/builder/src/components/backend/DataTable/buttons/TableFilterButton.svelte
+++ b/packages/builder/src/components/backend/DataTable/buttons/TableFilterButton.svelte
@@ -44,7 +44,14 @@
   }
 </script>
 
-<ActionButton icon="Filter" quiet {disabled} on:click={drawer.show} {selected}>
+<ActionButton
+  icon="Filter"
+  quiet
+  {disabled}
+  on:click={drawer.show}
+  {selected}
+  accentColor="#004EA6"
+>
   {text}
 </ActionButton>
 
diff --git a/packages/builder/src/components/backend/DataTable/buttons/grid/GridAutomationsButton.svelte b/packages/builder/src/components/backend/DataTable/buttons/grid/GridAutomationsButton.svelte
index 5216fb5ebb..a19dbe4696 100644
--- a/packages/builder/src/components/backend/DataTable/buttons/grid/GridAutomationsButton.svelte
+++ b/packages/builder/src/components/backend/DataTable/buttons/grid/GridAutomationsButton.svelte
@@ -39,9 +39,14 @@
 
 <DetailPopover title="Automations" minWidth={400} bind:this={popover}>
   <svelte:fragment slot="anchor" let:open>
-    <ActionButton icon="JourneyVoyager" selected={open} quiet
-      >Automations</ActionButton
+    <ActionButton
+      icon="JourneyVoyager"
+      selected={open || connectedAutomations.length}
+      quiet
+      accentColor="#5610AD"
     >
+      Automations
+    </ActionButton>
   </svelte:fragment>
   {#if !connectedAutomations.length}
     There aren't any automations connected to this data.
diff --git a/packages/builder/src/components/backend/DataTable/buttons/grid/GridColumnsSettingButton.svelte b/packages/builder/src/components/backend/DataTable/buttons/grid/GridColumnsSettingButton.svelte
index 8c66d9ecfc..3544a94d74 100644
--- a/packages/builder/src/components/backend/DataTable/buttons/grid/GridColumnsSettingButton.svelte
+++ b/packages/builder/src/components/backend/DataTable/buttons/grid/GridColumnsSettingButton.svelte
@@ -22,6 +22,7 @@
     on:click={() => (open = !open)}
     selected={open || anyRestricted}
     disabled={!$columns.length}
+    accentColor="#674D00"
   >
     {text}
   </ActionButton>
diff --git a/packages/builder/src/components/backend/DataTable/buttons/grid/GridRowActionsButton.svelte b/packages/builder/src/components/backend/DataTable/buttons/grid/GridRowActionsButton.svelte
index 52eee33453..90e446ce91 100644
--- a/packages/builder/src/components/backend/DataTable/buttons/grid/GridRowActionsButton.svelte
+++ b/packages/builder/src/components/backend/DataTable/buttons/grid/GridRowActionsButton.svelte
@@ -23,8 +23,8 @@
   $: tableId = ds?.tableId
   $: isView = ds?.type === "viewV2"
   $: fetchRowActions(tableId)
-  $: activeCount = 0
-  $: suffix = isView ? activeCount : rowActions.length
+  $: viewActiveCount = 0
+  $: activeCount = isView ? viewActiveCount : rowActions.length
 
   const rowActionUrl = derived([url, appStore], ([$url, $appStore]) => {
     return ({ automationId }) => {
@@ -62,8 +62,13 @@
 
 <DetailPopover title="Row actions" minWidth={400} maxWidth={400}>
   <svelte:fragment slot="anchor" let:open>
-    <ActionButton icon="Engagement" selected={open} quiet>
-      Row actions ({suffix})
+    <ActionButton
+      icon="Engagement"
+      selected={open || activeCount}
+      quiet
+      accentColor="#A24400"
+    >
+      Row actions ({activeCount})
     </ActionButton>
   </svelte:fragment>
   A row action is a user-triggered automation for a chosen row.
diff --git a/packages/builder/src/components/backend/DataTable/buttons/grid/GridScreensButton.svelte b/packages/builder/src/components/backend/DataTable/buttons/grid/GridScreensButton.svelte
index 48c9bdc96c..9baeccb353 100644
--- a/packages/builder/src/components/backend/DataTable/buttons/grid/GridScreensButton.svelte
+++ b/packages/builder/src/components/backend/DataTable/buttons/grid/GridScreensButton.svelte
@@ -19,7 +19,14 @@
 
 <DetailPopover title="Screens" minWidth={400}>
   <svelte:fragment slot="anchor" let:open>
-    <ActionButton icon="WebPage" selected={open} quiet>Screens</ActionButton>
+    <ActionButton
+      icon="WebPage"
+      selected={open || connectedScreens.length}
+      quiet
+      accentColor="#364800"
+    >
+      Screens
+    </ActionButton>
   </svelte:fragment>
   {#if !connectedScreens.length}
     There aren't any screens connected to this data.