From a0aa2e6e158ea8e77ee040e018bcca053e118da2 Mon Sep 17 00:00:00 2001
From: Alexander Platov <sas_lord@mail.ru>
Date: Tue, 7 Dec 2021 15:14:24 +0300
Subject: [PATCH] Update tooltip UI (#551)

Signed-off-by: Alexander Platov <sas_lord@mail.ru>
---
 packages/theme/styles/_colors.scss            |   2 +-
 packages/theme/styles/_layouts.scss           |   1 +
 packages/ui/src/components/Tooltip.svelte     |   4 +
 .../ui/src/components/TooltipInstance.svelte  | 194 +++++++++++++-----
 .../src/components/CommentPresenter.svelte    |  29 +--
 .../src/components/Workbench.svelte           |   8 +-
 6 files changed, 169 insertions(+), 69 deletions(-)

diff --git a/packages/theme/styles/_colors.scss b/packages/theme/styles/_colors.scss
index 7a1620634d..867676517a 100644
--- a/packages/theme/styles/_colors.scss
+++ b/packages/theme/styles/_colors.scss
@@ -97,7 +97,7 @@
   --theme-on-color: #4474F6;
   --theme-off-color: #77818E;
   --theme-bg-check: #F2F2F2;
-  --theme-tooltip-color: #2C2C34;
+  --theme-tooltip-color: #2F2F34;
   --theme-showmore-color: #484850;
 
   --theme-button-bg-enabled: #1F1F25;
diff --git a/packages/theme/styles/_layouts.scss b/packages/theme/styles/_layouts.scss
index 845ed108c6..431e382946 100644
--- a/packages/theme/styles/_layouts.scss
+++ b/packages/theme/styles/_layouts.scss
@@ -170,6 +170,7 @@ p:last-child { margin-block-end: 0; }
   align-items: center;
   flex-wrap: nowrap;
   white-space: nowrap;
+  width: fit-content;
   color: var(--theme-caption-color);
   cursor: pointer;
   .icon {
diff --git a/packages/ui/src/components/Tooltip.svelte b/packages/ui/src/components/Tooltip.svelte
index a4e807056f..bf32de2a50 100644
--- a/packages/ui/src/components/Tooltip.svelte
+++ b/packages/ui/src/components/Tooltip.svelte
@@ -38,3 +38,7 @@
 >
   <slot />
 </div>
+
+<style lang="scss">
+  .tooltip-trigger { width: fit-content; }
+</style>
diff --git a/packages/ui/src/components/TooltipInstance.svelte b/packages/ui/src/components/TooltipInstance.svelte
index 9703c60bdf..74cc5b8e42 100644
--- a/packages/ui/src/components/TooltipInstance.svelte
+++ b/packages/ui/src/components/TooltipInstance.svelte
@@ -14,18 +14,24 @@
 -->
 
 <script lang="ts">
+  import { afterUpdate, onDestroy } from 'svelte'
   import { tooltipstore as tooltip, closeTooltip } from '..'
   import type { TooltipAligment } from '..'
   import Label from './Label.svelte'
 
   let tooltipHTML: HTMLElement
+  let nubHTML: HTMLElement
   let dir: TooltipAligment
   let rect: DOMRect
   let rectAnchor: DOMRect
   let tooltipSW: boolean // tooltipSW = true - Label; false - Component
+  let nubDirection: 'top' | 'bottom' | 'left' | 'right' | undefined = undefined
+  let clWidth: number
+  let clHeight: number
 
   $: tooltipSW = $tooltip.component ? false : true
-  $: {
+
+  const fitTooltip = (): void => {
     if (($tooltip.label || $tooltip.component) && tooltipHTML) {
       if ($tooltip.element) {
         const doc = document.body.getBoundingClientRect()
@@ -35,30 +41,39 @@
 
         if ($tooltip.component) {
 
-          if (rectAnchor.bottom + tooltipHTML.clientHeight + 28 < doc.height) {
-            tooltipHTML.style.top = `calc(${rectAnchor.bottom}px + .75rem)`
+          if (rect.bottom + tooltipHTML.clientHeight + 28 < doc.height) {
+            tooltipHTML.style.top = `calc(${rect.bottom}px + 5px + .25rem)`
             dir = 'bottom'
-          } else if (rectAnchor.top > doc.height - rectAnchor.bottom) {
-            tooltipHTML.style.bottom = `calc(${doc.height - rectAnchor.y}px + .75rem)`
-            if (tooltipHTML.clientHeight > rectAnchor.top - 28) {
+          } else if (rect.top > doc.height - rect.bottom) {
+            tooltipHTML.style.bottom = `calc(${doc.height - rect.y}px + 5px + .25rem)`
+            if (tooltipHTML.clientHeight > rect.top - 28) {
               tooltipHTML.style.top = '1rem'
-              tooltipHTML.style.height = rectAnchor.top - 28 + 'px'
+              tooltipHTML.style.height = rect.top - 28 + 'px'
             }
             dir = 'top'
           } else {
-            tooltipHTML.style.top = `calc(${rectAnchor.bottom}px + .75rem)`
-            if (tooltipHTML.clientHeight > doc.height - rectAnchor.bottom - 28) {
+            tooltipHTML.style.top = `calc(${rect.bottom}px + 5px + .25rem)`
+            if (tooltipHTML.clientHeight > doc.height - rect.bottom - 28) {
               tooltipHTML.style.bottom = '1rem'
-              tooltipHTML.style.height = doc.height - rectAnchor.bottom - 28 + 'px'
+              tooltipHTML.style.height = doc.height - rect.bottom - 28 + 'px'
             }
             dir = 'bottom'
           }
-          if (rectAnchor.left + tooltipHTML.clientWidth + 16 > doc.width) {
-            tooltipHTML.style.left = ''
-            tooltipHTML.style.right = doc.width - rectAnchor.right + 'px'
-          } else {
-            tooltipHTML.style.left = rectAnchor.left + 'px'
-            tooltipHTML.style.right = ''
+
+          if (rect.width / 2 + rect.left + clWidth / 2 > doc.width - 8)
+            tooltipHTML.style.right = '.5rem'
+          else if (rect.width / 2 + rect.left - clWidth / 2 < 8)
+            tooltipHTML.style.left = '.5rem'
+          else if ((rect.width / 2 + rect.left + clWidth / 2 < doc.width) && (rect.width / 2 + rect.left - clWidth / 2 > 0))
+            tooltipHTML.style.left = rect.width / 2 + rect.left - clWidth / 2 + 'px'
+          else tooltipHTML.style.left = tooltipHTML.style.right = '.5rem'
+
+          if (nubHTML) {
+            nubHTML.style.top = rect.top + 'px'
+            nubHTML.style.left = rect.left + 'px'
+            nubHTML.style.width = rect.width + 'px'
+            nubHTML.style.height = rect.height + 'px'
+            nubDirection = dir
           }
 
         } else {
@@ -122,16 +137,24 @@
       }
     }
   }
+
+  afterUpdate(() => fitTooltip())
+  onDestroy(() => hideTooltip())
 </script>
 
-<svelte:window on:mousemove={(ev) => { whileShow(ev) }} />
+<svelte:window on:resize={fitTooltip} on:mousemove={(ev) => { whileShow(ev) }} />
+<svg class="mask">
+  <clipPath id="nub-bg"><path d="M7.3.6 4.2 4.3C2.9 5.4 1.5 6 0 6v1h18V6c-1.5 0-2.9-.6-4.2-1.7L10.7.6C9.9-.1 8.5-.2 7.5.4c0 .1-.1.1-.2.2z" /></clipPath>
+  <clipPath id="nub-border"><path d="M4.8 5.1 8 1.3s.1 0 .1-.1c.5-.3 1.4-.3 1.9.1L13.1 5l.1.1 1.2.9H18c-1.5 0-2.9-.6-4.2-1.7L10.7.6C9.9-.1 8.5-.2 7.5.4c0 .1-.1.1-.2.2L4.2 4.3C2.9 5.4 1.5 6 0 6h3.6l1.2-.9z" /></clipPath>
+</svg>
 {#if $tooltip.component}
-  <div class="popup" bind:this={tooltipHTML}>
+  <div class="popup" bind:clientWidth={clWidth} bind:clientHeight={clHeight} bind:this={tooltipHTML}>
     {#if $tooltip.label}<div class="header"><Label label={$tooltip.label} /></div>{/if}
     <svelte:component this={$tooltip.component} {...$tooltip.props} />
   </div>
+  <div bind:this={nubHTML} class="nub {nubDirection ?? ''}" />
 {:else if $tooltip.label}
-  <div class="tooltip {dir}" bind:this={tooltipHTML}>
+  <div class="tooltip {dir ?? ''}" bind:this={tooltipHTML}>
     <Label label={$tooltip.label} />
   </div>
 {/if}
@@ -148,85 +171,146 @@
     position: fixed;
     display: flex;
     flex-direction: column;
-    padding: 1.5rem;
+    max-width: 30rem;
+    padding: 1rem;
     color: var(--theme-caption-color);
-    background-color: var(--theme-button-bg-focused);
-    border: 1px solid var(--theme-button-border-enabled);
+    background-color: var(--theme-tooltip-color);
+    border: 1px solid var(--theme-bg-accent-color);
     border-radius: .75rem;
+    box-shadow: 0px 10px 20px rgba(0, 0, 0, .2);
     user-select: none;
-    filter: drop-shadow(0 1.5rem 4rem rgba(0, 0, 0, .35));
     z-index: 10000;
   }
 
+  .nub {
+    position: fixed;
+    // background-color: rgba(255, 255, 0, .5);
+    user-select: none;
+    z-index: 10000;
+
+    &::after, &::before {
+      position: absolute;
+      width: 18px;
+      height: 7px;
+    }
+    &::before {
+      background-color: var(--theme-tooltip-color);
+      clip-path: url("#nub-bg");
+      z-index: 1;
+    }
+    &::after {
+      background-color: var(--theme-bg-accent-color);
+      clip-path: url("#nub-border");
+      z-index: 2;
+    }
+
+    &.top::after, &.bottom::after,
+    &.top::before, &.bottom::before,
+    &.right::after, &.left::after,
+    &.right::before, &.left::before { content: ''; }
+    &.top::after, &.bottom::after,
+    &.top::before, &.bottom::before {
+      left: 50%;
+      margin-left: -9px;
+    }
+    &.top::after, &.top::before {
+      top: calc(-7px - .25rem);
+      transform: rotate(180deg);
+    }
+    &.bottom::after, &.bottom::before {
+      bottom: calc(-7px - .25rem);
+    }
+
+    &.right::after, &.left::after,
+    &.right::before, &.left::before {
+      top: 50%;
+      margin-top: -9px;
+    }
+    &.left::after, &.left::before {
+      transform-origin: left top;
+      left: -.25rem;
+      transform: rotate(90deg);
+    }
+    &.right::after, &.right::before {
+      transform-origin: right top;
+      right: -.25rem;
+      transform: rotate(-90deg);
+    }
+  }
+
   .tooltip {
     position: fixed;
-    padding: .5rem;
+    padding: .5rem .75rem;
+    text-align: center;
     color: var(--theme-caption-color);
     background-color: var(--theme-tooltip-color);
     border: 1px solid var(--theme-bg-accent-color);
-    border-radius: .5rem;
+    border-radius: .75rem;
+    box-shadow: 0px 10px 20px rgba(0, 0, 0, .2);
     user-select: none;
-    text-align: center;
     z-index: 10000;
 
-    &::after {
+    &::after, &::before {
       content: '';
       position: absolute;
-      width: .875rem;
-      height: .875rem;
+      width: 18px;
+      height: 7px;
+    }
+    &::before {
       background-color: var(--theme-tooltip-color);
-      border: 1px solid var(--theme-bg-accent-color);
-      border-radius: 0 0 3px;
-      clip-path: polygon(100% 25%, 100% 100%, 25% 100%);
+      clip-path: url("#nub-bg");
+      z-index: 1;
+    }
+    &::after {
+      background-color: var(--theme-bg-accent-color);
+      clip-path: url("#nub-border");
+      z-index: 2;
     }
 
-    &.top::after,
-    &.bottom::after {
+    &.top::after, &.bottom::after,
+    &.top::before, &.bottom::before {
       left: 50%;
-      margin-left: -.5rem;
+      margin-left: -9px;
     }
     &.top {
       bottom: 100%;
-      box-shadow: 0px 8px 20px rgba(0, 0, 0, .35);
-      &::after {
-        bottom: -.3125rem;
-        transform: rotate(45deg);
+      &::after, &::before {
+        bottom: -7px;
+        transform: rotate(180deg);
       }
     }
     &.bottom {
       top: 100%;
-      box-shadow: 0px -8px 20px rgba(0, 0, 0, .35);
-      &::after {
-        top: -.3125rem;
-        transform: rotate(-135deg);
+      &::after, &::before {
+        top: -7px;
       }
     }
 
-    &.right::after,
-    &.left::after {
+    &.right::after, &.left::after,
+    &.right::before, &.left::before {
       top: 50%;
-      margin-top: -.5rem;
+      margin-top: -9px;
     }
     &.right {
       left: 100%;
-      box-shadow: -8px 0px 20px rgba(0, 0, 0, .35);
-      &::after {
-        left: -.3125rem;
-        transform: rotate(135deg);
+      &::after, &::before {
+        transform-origin: right top;
+        left: -25px;
+        transform: rotate(-90deg);
       }
     }
     &.left {
       right: 100%;
-      box-shadow: 8px 0px 20px rgba(0, 0, 0, .35);
-      &::after {
-        right: -.3125rem;
-        transform: rotate(-45deg);
+      &::after, &::before {
+        transform-origin: left top;
+        right: -25px;
+        transform: rotate(90deg);
       }
     }
   }
   .no-arrow {
     box-shadow: 0px 0px 20px rgba(0, 0, 0, .75);
-    &::after {
+    &::after, &::before {
       content: none;
     }
   }
diff --git a/plugins/chunter-resources/src/components/CommentPresenter.svelte b/plugins/chunter-resources/src/components/CommentPresenter.svelte
index cf289878e6..8591313a44 100644
--- a/plugins/chunter-resources/src/components/CommentPresenter.svelte
+++ b/plugins/chunter-resources/src/components/CommentPresenter.svelte
@@ -39,22 +39,27 @@
       </div>
       <div class="content-trans-color ml-4"><TimeSince value={value.modifiedOn} /></div>
     </div>
-    <div class="content-color">
+    <div class="comment-content">
       <MessageViewer message={value.message}/>
     </div>
   </div>
 </div>
 
 <style lang="scss">
-    .avatar {
-      margin-right: 1rem;
-      min-width: 2.25rem;
-    }
-
-    .header {
-      display: inline-flex;
-      justify-content: space-between;
-      align-items: baseline;
-      margin-bottom: .25rem;
-    }
+  .avatar {
+    margin-right: 1rem;
+    min-width: 2.25rem;
+  }
+  .header {
+    display: inline-flex;
+    justify-content: space-between;
+    align-items: baseline;
+    margin-bottom: .25rem;
+  }
+  .comment-content {
+    display: -webkit-box;
+    -webkit-line-clamp: 7;
+    -webkit-box-orient: vertical;  
+    overflow: hidden;
+  }
 </style>
\ No newline at end of file
diff --git a/plugins/workbench-resources/src/components/Workbench.svelte b/plugins/workbench-resources/src/components/Workbench.svelte
index 4841a74868..0c57b77c86 100644
--- a/plugins/workbench-resources/src/components/Workbench.svelte
+++ b/plugins/workbench-resources/src/components/Workbench.svelte
@@ -29,7 +29,7 @@
   import SpaceHeader from './SpaceHeader.svelte'
   import SpaceView from './SpaceView.svelte'
   
-  import { AnyComponent, Component, location, Popup, showPopup, TooltipInstance, closeTooltip } from '@anticrm/ui'
+  import { AnyComponent, Component, location, Popup, showPopup, TooltipInstance, closeTooltip, ActionIcon, IconEdit } from '@anticrm/ui'
   import core from '@anticrm/core'
   import AccountPopup from './AccountPopup.svelte'
   import AppItem from './AppItem.svelte'
@@ -116,6 +116,12 @@
         <NavHeader label={currentApplication.label} />
       {/if}
       <Navigator model={navigatorModel} />
+      <!-- <div class="flex-center safari-gap-2" style="height: 20rem">
+        <ActionIcon icon={IconEdit} label={workbench.string.ShowMenu} size={'small'} direction={'left'} />
+        <ActionIcon icon={IconEdit} label={workbench.string.ShowMenu} size={'small'} direction={'top'} />
+        <ActionIcon icon={IconEdit} label={workbench.string.ShowMenu} size={'small'} direction={'bottom'} />
+        <ActionIcon icon={IconEdit} label={workbench.string.ShowMenu} size={'small'} direction={'right'} />
+      </div> -->
     </div>
     {/if}
     <div class="panel-component">