Merge pull request #8944 from Budibase/side-panel-tweaks

Side panel tweaks

packages/bbui/src/Skeleton/Skeleton.svelte

@@ -9,7 +9,7 @@
     height: 100%;
     width: 100%;
     opacity: 0;
-    background-color: var(--spectrum-global-color-gray-300) !important;
+    background-color: var(--spectrum-global-color-gray-200) !important;
     border-radius: 7px;
     overflow: hidden;
     position: relative;
@@ -31,8 +31,8 @@
     background-image: linear-gradient(
       90deg,
       rgba(255, 255, 255, 0) 0,
-      rgba(255, 255, 255, 0.2) 20%,
+      rgba(255, 255, 255, 0.15) 20%,
-      rgba(255, 255, 255, 0.5) 60%,
+      rgba(255, 255, 255, 0.3) 60%,
       rgba(255, 255, 255, 0)
     );
     animation: shimmer 2s infinite;
@@ -44,7 +44,7 @@
       opacity: 0;
     }
     100% {
-      opacity: 1;
+      opacity: 0.75;
     }
   }
 

packages/client/src/components/app/SidePanel.svelte

@@ -57,7 +57,9 @@
   class="side-panel"
   class:open
 >
-  <slot />
+  {#if $sidePanelStore.open}
+    <slot />
+  {/if}
 </div>
 
 <style>

packages/client/src/stores/sidePanel.js

@@ -11,18 +11,25 @@ export const createSidePanelStore = () => {
       open: $store.contentId != null,
     }
   })
+  let timeout
 
   const open = id => {
+    clearTimeout(timeout)
     store.update(state => {
       state.contentId = id
       return state
     })
   }
+
+  // Delay closing by 50ms to avoid toggling visibility when cycling though
+  // records
   const close = () => {
-    store.update(state => {
+    timeout = setTimeout(() => {
-      state.contentId = null
+      store.update(state => {
-      return state
+        state.contentId = null
-    })
+        return state
+      })
+    }, 50)
   }
 
   return {