From 75c4b927662bc1b274ced14a600f717affdd67b5 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <andrew@kingston.dev>
Date: Wed, 5 Jul 2023 15:46:07 +0100
Subject: [PATCH] Add new tooltip which is far better

---
 packages/bbui/src/Tooltip/AbsTooltip.svelte   | 120 ++++++++++++++++++
 packages/bbui/src/index.js                    |   5 +
 .../components/grid/layout/HeaderRow.svelte   |  25 +++-
 3 files changed, 143 insertions(+), 7 deletions(-)
 create mode 100644 packages/bbui/src/Tooltip/AbsTooltip.svelte

diff --git a/packages/bbui/src/Tooltip/AbsTooltip.svelte b/packages/bbui/src/Tooltip/AbsTooltip.svelte
new file mode 100644
index 0000000000..a407c34de3
--- /dev/null
+++ b/packages/bbui/src/Tooltip/AbsTooltip.svelte
@@ -0,0 +1,120 @@
+<script context="module">
+  export const TooltipDirection = {
+    Top: "top",
+    Right: "right",
+    Bottom: "bottom",
+    Left: "left",
+  }
+
+  export const TooltipType = {
+    Default: "default",
+    Info: "info",
+    Positive: "positive",
+    Negative: "negative",
+  }
+</script>
+
+<script>
+  import Portal from "svelte-portal"
+  import { fade } from "svelte/transition"
+  import "@spectrum-css/tooltip/dist/index-vars.css"
+
+  export let direction = TooltipDirection.Top
+  export let type = TooltipType.Positive
+  export let text = ""
+
+  let wrapper
+  let visible = false
+  let left = 0
+  let top = 0
+
+  const show = () => {
+    const node = wrapper?.children?.[0]
+    if (!node) {
+      return
+    }
+    const bounds = node.getBoundingClientRect()
+
+    if (direction === TooltipDirection.Top) {
+      left = bounds.left + bounds.width / 2
+      top = bounds.top
+    } else if (direction === TooltipDirection.Right) {
+      left = bounds.left + bounds.width
+      top = bounds.top + bounds.height / 2
+    } else if (direction === TooltipDirection.Bottom) {
+      left = bounds.left + bounds.width / 2
+      top = bounds.top + bounds.height
+    } else if (direction === TooltipDirection.Left) {
+      left = bounds.left
+      top = bounds.top + bounds.height / 2
+    } else {
+      return
+    }
+
+    visible = true
+  }
+  const hide = () => {
+    visible = false
+  }
+</script>
+
+<div
+  bind:this={wrapper}
+  class="abs-tooltip"
+  on:mouseover={show}
+  on:mouseleave={hide}
+>
+  <slot />
+</div>
+{#if visible}
+  <Portal target=".spectrum">
+    <span
+      class="spectrum-Tooltip spectrum-Tooltip--{type} spectrum-Tooltip--{direction} is-open"
+      style="left:{left}px;top:{top}px;"
+      transition:fade|local={{ duration: 130 }}
+    >
+      <span class="spectrum-Tooltip-label">{text}</span>
+      <span class="spectrum-Tooltip-tip" />
+    </span>
+  </Portal>
+{/if}
+
+<style>
+  .abs-tooltip {
+    display: contents;
+  }
+  .spectrum-Tooltip {
+    position: absolute;
+    z-index: 999;
+    pointer-events: none;
+    margin: 0;
+    max-width: 280px;
+  }
+  .spectrum-Tooltip-label {
+    text-overflow: ellipsis;
+    white-space: nowrap;
+    overflow: hidden;
+  }
+
+  /* Colour overrides for default type */
+  .spectrum-Tooltip--default {
+    background: var(--spectrum-global-color-gray-500);
+  }
+  .spectrum-Tooltip--default .spectrum-Tooltip-tip {
+    border-top-color: var(--spectrum-global-color-gray-500);
+  }
+
+  /* Direction styles */
+  .spectrum-Tooltip--top {
+    transform: translateX(-50%) translateY(calc(-100% - 8px));
+  }
+  .spectrum-Tooltip--right {
+    transform: translateX(8px) translateY(-50%);
+  }
+  .spectrum-Tooltip--bottom {
+    transform: translateX(-50%) translateY(8px);
+  }
+  .spectrum-Tooltip--left {
+    transform: translateX(calc(-100% - 8px)) translateY(-50%);
+  }
+</style>
diff --git a/packages/bbui/src/index.js b/packages/bbui/src/index.js
index d26b938dd5..cbe9ddbff5 100644
--- a/packages/bbui/src/index.js
+++ b/packages/bbui/src/index.js
@@ -36,6 +36,11 @@ export { default as Layout } from "./Layout/Layout.svelte"
 export { default as Page } from "./Layout/Page.svelte"
 export { default as Link } from "./Link/Link.svelte"
 export { default as Tooltip } from "./Tooltip/Tooltip.svelte"
+export {
+  default as AbsTooltip,
+  TooltipDirection,
+  TooltipType,
+} from "./Tooltip/AbsTooltip.svelte"
 export { default as TooltipWrapper } from "./Tooltip/TooltipWrapper.svelte"
 export { default as Menu } from "./Menu/Menu.svelte"
 export { default as MenuSection } from "./Menu/Section.svelte"
diff --git a/packages/frontend-core/src/components/grid/layout/HeaderRow.svelte b/packages/frontend-core/src/components/grid/layout/HeaderRow.svelte
index f31cc198fb..de73281bd5 100644
--- a/packages/frontend-core/src/components/grid/layout/HeaderRow.svelte
+++ b/packages/frontend-core/src/components/grid/layout/HeaderRow.svelte
@@ -2,7 +2,12 @@
   import { getContext } from "svelte"
   import GridScrollWrapper from "./GridScrollWrapper.svelte"
   import HeaderCell from "../cells/HeaderCell.svelte"
-  import { Icon } from "@budibase/bbui"
+  import {
+    Icon,
+    AbsTooltip,
+    TooltipType,
+    TooltipDirection,
+  } from "@budibase/bbui"
 
   const {
     renderedColumns,
@@ -30,13 +35,19 @@
     </div>
   </GridScrollWrapper>
   {#if $config.allowSchemaChanges}
-    <div
-      class="add"
-      style="left:{left}px"
-      on:click={() => dispatch("add-column")}
+    <AbsTooltip
+      text="Click here to create your first column"
+      direction={TooltipDirection.Bottom}
+      type={TooltipType.Info}
     >
-      <Icon name="Add" />
-    </div>
+      <div
+        class="add"
+        style="left:{left}px"
+        on:click={() => dispatch("add-column")}
+      >
+        <Icon name="Add" />
+      </div>
+    </AbsTooltip>
   {/if}
 </div>