From 609cef854ee327cfe644b9a79b3df493f7e20e3c Mon Sep 17 00:00:00 2001
From: Enoch Riese <enoch.riese@gmail.com>
Date: Wed, 21 Jun 2023 23:11:14 -0500
Subject: [PATCH] sticky draft toolbar with tooltips

---
 sites/shared/components/workbench/header.mjs  |  4 +--
 .../workbench/views/draft/index.mjs           |  2 +-
 .../workbench/views/view-header.mjs           | 34 ++++++++++++-------
 .../components/workbench/workbench.en.yaml    |  2 +-
 4 files changed, 26 insertions(+), 16 deletions(-)

diff --git a/sites/shared/components/workbench/header.mjs b/sites/shared/components/workbench/header.mjs
index ea0fab28017..621d10b0d04 100644
--- a/sites/shared/components/workbench/header.mjs
+++ b/sites/shared/components/workbench/header.mjs
@@ -160,8 +160,8 @@ export const WorkbenchHeader = ({ view, setView }) => {
     >
       <div
         className={`
-      flex flex-col lg:justify-between
-      items-center w-full sticky lg:top-26`}
+      flex flex-col
+      items-center w-full sticky top-4 lg:top-28`}
       >
         <NavIcons {...{ setView, setDense, dense, view }} />
       </div>
diff --git a/sites/shared/components/workbench/views/draft/index.mjs b/sites/shared/components/workbench/views/draft/index.mjs
index 663f579f560..7feaa189187 100644
--- a/sites/shared/components/workbench/views/draft/index.mjs
+++ b/sites/shared/components/workbench/views/draft/index.mjs
@@ -3,7 +3,7 @@ import { DraftMenu, ns as menuNs } from './menu.mjs'
 import { ViewHeader, ns as headerNs } from 'shared/components/workbench/views/view-header.mjs'
 import { PanZoomContextProvider } from 'shared/components/workbench/pattern/pan-zoom-context.mjs'
 
-export const ns = menuNs
+export const ns = [menuNs, ...headerNs]
 
 export const DraftView = ({
   design,
diff --git a/sites/shared/components/workbench/views/view-header.mjs b/sites/shared/components/workbench/views/view-header.mjs
index f376e699f46..a6529edfa2d 100644
--- a/sites/shared/components/workbench/views/view-header.mjs
+++ b/sites/shared/components/workbench/views/view-header.mjs
@@ -12,7 +12,7 @@ import {
   ClearIcon,
 } from 'shared/components/icons.mjs'
 
-export const ns = ['common']
+export const ns = ['common', 'core-settings', 'ui-settings']
 
 const ZoomInIcon = (props) => (
   <IconWrapper {...props}>
@@ -27,15 +27,17 @@ const ZoomOutIcon = (props) => (
 )
 
 const IconButton = ({ Icon, onClick, dflt = true, title, hide = false }) => (
-  <button
-    onClick={onClick}
-    className={`text-${dflt ? 'neutral-content' : 'accent'} hover:text-secondary-focus ${
-      hide ? 'invisible' : ''
-    }`}
-    title={title}
-  >
-    <Icon />
-  </button>
+  <div className="tooltip tooltip-bottom tooltip-primary flex items-center" data-tip={title}>
+    <button
+      onClick={onClick}
+      className={`text-${dflt ? 'neutral-content' : 'accent'} hover:text-secondary-focus ${
+        hide ? 'invisible' : ''
+      }`}
+      title={title}
+    >
+      <Icon />
+    </button>
+  </div>
 )
 
 const ZoomButtons = ({ t }) => {
@@ -71,7 +73,7 @@ const Spacer = () => <span className="opacity-50">|</span>
 export const ViewHeader = ({ update, settings, ui, control, setSettings }) => {
   const { t } = useTranslation(ns)
   return (
-    <div className="flex flex-row flex-wrap gap-4 py-4 mt-2 pt-4 w-full bg-neutral text-neutral-content items-center justify-center">
+    <div className="flex flex-row flex-wrap gap-4 py-4 pt-4 w-full bg-neutral text-neutral-content items-center justify-center lg:sticky top-0 z-20">
       <ZoomButtons t={t} />
       <Spacer />
       <div className="flex flex-row items-center gap-4">
@@ -79,11 +81,13 @@ export const ViewHeader = ({ update, settings, ui, control, setSettings }) => {
           Icon={SaIcon}
           dflt={settings.sabool ? false : true}
           onClick={() => update.toggleSa()}
+          title={t('core-settings:sabool.t')}
         />
         <IconButton
           Icon={PaperlessIcon}
           dflt={settings.paperless ? false : true}
           onClick={() => update.settings(['paperless'], !settings.paperless)}
+          title={t('core-settings:paperless.t')}
         />
         <IconButton
           Icon={DetailIcon}
@@ -94,6 +98,7 @@ export const ViewHeader = ({ update, settings, ui, control, setSettings }) => {
               typeof settings.complete === 'undefined' ? 0 : settings.complete ? 0 : 1
             )
           }
+          title={t('core-settings:complete.t')}
         />
         <IconButton
           Icon={
@@ -105,10 +110,14 @@ export const ViewHeader = ({ update, settings, ui, control, setSettings }) => {
           onClick={() =>
             update.settings(['units'], settings.units === 'imperial' ? 'metric' : 'imperial')
           }
+          title={t('core-settings:units.t')}
         />
       </div>
       <Spacer />
-      <div className="flex flex-row items-center">
+      <div
+        className="tooltip tooltip-primary tooltip-bottom flex flex-row items-center"
+        data-tip={t('ui-settings:control.t')}
+      >
         {[1, 2, 3, 4, 5].map((score) => (
           <button onClick={() => update.setControl(score)} className="text-primary" key={score}>
             <BulletIcon fill={control >= score ? true : false} />
@@ -121,6 +130,7 @@ export const ViewHeader = ({ update, settings, ui, control, setSettings }) => {
           Icon={RocketIcon}
           dflt={ui.renderer !== 'svg'}
           onClick={() => update.ui(['renderer'], ui.renderer === 'react' ? 'svg' : 'react')}
+          title={t('ui-settings:renderer.t')}
         />
       </div>
       <Spacer />
diff --git a/sites/shared/components/workbench/workbench.en.yaml b/sites/shared/components/workbench/workbench.en.yaml
index 52eb3766e64..9b2500d65c4 100644
--- a/sites/shared/components/workbench/workbench.en.yaml
+++ b/sites/shared/components/workbench/workbench.en.yaml
@@ -30,4 +30,4 @@ patternLogs: Pattern logs
 patternInspector: Pattern Inspector
 docs: Documentation
 configurePattern: Configure pattern
-measies: Measurements
+measies: Pattern Measurements