[semver:patch] Fix panel opening states after pre-loading

strigo · Oct 31, 2022 · c112b3b · c112b3b
1 parent 86155d2
commit c112b3b
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 24 deletions.
diff --git a/dist/development/strigo.sdk.js b/dist/development/strigo.sdk.js
@@ -13411,7 +13411,10 @@ ${JSON.stringify(parsedContext)}` : "");
       case "rendered" /* RENDERED */: {
         LoggerInstance.info("Panel rendered message received");
         setSessionValue("isRendered", true);
-        window.Strigo?.expandPanel();
+        removeLoader();
+        if (shouldPanelBeOpen()) {
+          window.Strigo?.expandPanel();
+        }
         break;
       }
       default: {
@@ -13677,9 +13680,10 @@ ${JSON.stringify(parsedContext)}` : "");
         this.config.configured = true;
         LoggerInstance.info("Finished SDK setup.");
         if (openWidget2) {
+          const shouldPanelBeOpen2 = shouldPanelBeOpen();
           this.open();
-          setSessionValue("shouldPanelBeOpen", false);
           this.collapse();
+          setSessionValue("shouldPanelBeOpen", shouldPanelBeOpen2);
         }
       } catch (err) {
         LoggerInstance.error("Could not setup SDK", { err });

diff --git a/dist/development/styles/strigo-academy-hat.css b/dist/development/styles/strigo-academy-hat.css
@@ -12,7 +12,7 @@
   border-radius: 50%;
   display: inline-block;
   box-sizing: border-box;
-  animation: rotation 0.6s linear infinite;
+  animation: rotation 0.5s linear infinite;
 }
 @keyframes rotation {
   0% {
@@ -32,7 +32,7 @@
   align-items: center;
   justify-content: center;
   cursor: pointer;
-  box-shadow: 0px 2px 1px rgba(0, 0, 0, 0.12), 0px 5px 8px rgba(0, 0, 0, 0.26);
+  box-shadow: 0px 2px 2px rgba(0, 0, 0, 0.2);
   border-radius: 8px 0px 0px 8px;
   background-color: var(--customizable-hat-bg-color);
   z-index: 9999999999;

diff --git a/dist/production/strigo.sdk.min.js b/dist/production/strigo.sdk.min.js
diff --git a/dist/production/styles/strigo-academy-hat.min.css b/dist/production/styles/strigo-academy-hat.min.css
diff --git a/src/modules/listeners/listeners.ts b/src/modules/listeners/listeners.ts
@@ -3,7 +3,7 @@ import * as configManager from '../config/config';
 import overlayWidget from '../widgets/overlay';
 import { Logger } from '../../services/logger';
 import { WidgetFlavors } from '../widgets/widget.types';
-import { openWidget } from '../document/document';
+import { openWidget, removeLoader } from '../document/document';
 
 import { EventTypes, MessageTypes } from './listeners.types';
 
@@ -82,6 +82,7 @@ function onHostEventHandler(ev: MessageEvent<unknown>): void {
     case MessageTypes.RENDERED: {
       Logger.info('Panel rendered message received');
       sessionManager.setSessionValue('isRendered', true);
+      removeLoader();
 
       if (sessionManager.shouldPanelBeOpen()) {
         window.Strigo?.expandPanel();

diff --git a/src/modules/session/session.ts b/src/modules/session/session.ts
@@ -27,8 +27,8 @@ export function getSession(): StrigoSession {
 }
 
 export function shouldPanelBeOpen(): boolean {
+
   return getSession() ? getSession().shouldPanelBeOpen : true;
-
 }
 
 export function getWidgetFlavor(): WidgetFlavors {

diff --git a/src/strigo/index.ts b/src/strigo/index.ts
@@ -24,7 +24,6 @@ class StrigoSDK implements IStrigoSDK {
   init(): void {
     try {
       Logger.info('Initializing SDK...');
-      sessionManager.setSessionValue('shouldPanelBeOpen', true);
 
       if (this.config.initialized) {
         Logger.info('SDK was already initialized');
@@ -150,10 +149,14 @@ class StrigoSDK implements IStrigoSDK {
       Logger.info('Finished SDK setup.');
 
       if (openWidget) {
+        // Save the original state of the panel
+        const shouldPanelBeOpen = sessionManager.shouldPanelBeOpen();
         this.open();
-
         // Collapse the panel so it would open when fully loaded
         this.collapse();
+
+        // Return the original state of the panel after collapsing
+        sessionManager.setSessionValue('shouldPanelBeOpen', shouldPanelBeOpen);
       }
     } catch (err) {
       Logger.error('Could not setup SDK', { err });
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,8 +27,8 @@ export function getSession(): StrigoSession { @@
     }
     export function shouldPanelBeOpen(): boolean {
       return getSession() ? getSession().shouldPanelBeOpen : true;
     }
     export function getWidgetFlavor(): WidgetFlavors {
@@ Expand Down @@