From 1107328443b5e0bc29a43ebf2ee01d50d38656bf Mon Sep 17 00:00:00 2001
From: Kristina <kristin.fefelova@gmail.com>
Date: Wed, 9 Oct 2024 08:46:31 +0400
Subject: [PATCH] Fix video autoclose (#6843)

Signed-off-by: Kristina Fefelova <kristin.fefelova@gmail.com>
---
 .../src/components/chat/navigator/NavItem.svelte     |  2 +-
 .../love-resources/src/components/ControlExt.svelte  | 11 ++++++++---
 .../src/components/NotifyMarker.svelte               | 12 +++++++++++-
 .../src/components/sidebar/widgets/WidgetsBar.svelte |  2 +-
 plugins/workbench-resources/src/sidebar.ts           | 12 ++++++++++--
 5 files changed, 31 insertions(+), 8 deletions(-)

diff --git a/plugins/chunter-resources/src/components/chat/navigator/NavItem.svelte b/plugins/chunter-resources/src/components/chat/navigator/NavItem.svelte
index 19189fc5e6..aba8a60fec 100644
--- a/plugins/chunter-resources/src/components/chat/navigator/NavItem.svelte
+++ b/plugins/chunter-resources/src/components/chat/navigator/NavItem.svelte
@@ -102,7 +102,7 @@
       <div class="antiHSpacer" />
     {:else if secondaryNotifyMarker}
       <div class="antiHSpacer" />
-      <NotifyMarker count={0} kind="simple" />
+      <NotifyMarker count={0} kind="with-dot" />
       <div class="antiHSpacer" />
     {/if}
   </svelte:fragment>
diff --git a/plugins/love-resources/src/components/ControlExt.svelte b/plugins/love-resources/src/components/ControlExt.svelte
index ae7499f9f0..223055ceb0 100644
--- a/plugins/love-resources/src/components/ControlExt.svelte
+++ b/plugins/love-resources/src/components/ControlExt.svelte
@@ -301,7 +301,8 @@
   }
 
   function checkActiveVideo (loc: Location, video: boolean, room: Ref<Room> | undefined): void {
-    const isOpened = $sidebarStore.widgetsState.has(love.ids.VideoWidget)
+    const widgetState = $sidebarStore.widgetsState.get(love.ids.VideoWidget)
+    const isOpened = widgetState !== undefined
 
     if (room === undefined) {
       if (isOpened) {
@@ -319,11 +320,15 @@
           {
             room
           },
-          loc.path[2] !== loveId
+          { active: loc.path[2] !== loveId, openedByUser: false }
         )
       }
 
-      if (loc.path[2] === loveId && $sidebarStore.widget === love.ids.VideoWidget) {
+      if (
+        loc.path[2] === loveId &&
+        $sidebarStore.widget === love.ids.VideoWidget &&
+        widgetState?.openedByUser !== true
+      ) {
         minimizeSidebar()
       }
     } else {
diff --git a/plugins/notification-resources/src/components/NotifyMarker.svelte b/plugins/notification-resources/src/components/NotifyMarker.svelte
index b9cf69670a..e4e5e19469 100644
--- a/plugins/notification-resources/src/components/NotifyMarker.svelte
+++ b/plugins/notification-resources/src/components/NotifyMarker.svelte
@@ -14,7 +14,7 @@
 -->
 <script lang="ts">
   export let count: number = 0
-  export let kind: 'primary' | 'simple' = 'primary'
+  export let kind: 'primary' | 'simple' | 'with-dot' = 'primary'
   export let size: 'xx-small' | 'x-small' | 'small' | 'medium' = 'small'
 
   const maxNumber = 9
@@ -34,6 +34,10 @@
   <div class="notifyMarker {size} {kind}" />
 {/if}
 
+{#if kind === 'with-dot'}
+  <div class="notifyMarker {size} {kind}">●</div>
+{/if}
+
 <style lang="scss">
   .notifyMarker {
     display: flex;
@@ -44,6 +48,7 @@
     font-weight: 700;
 
     &.simple,
+    &.with-dot,
     &.primary {
       background-color: var(--global-higlight-Color);
       color: var(--global-on-accent-TextColor);
@@ -70,5 +75,10 @@
       height: 1.25rem;
       font-size: 0.625rem;
     }
+
+    &.with-dot {
+      font-weight: 400;
+      font-size: 0.25rem;
+    }
   }
 </style>
diff --git a/plugins/workbench-resources/src/components/sidebar/widgets/WidgetsBar.svelte b/plugins/workbench-resources/src/components/sidebar/widgets/WidgetsBar.svelte
index 5f5717b4db..9bd512250a 100644
--- a/plugins/workbench-resources/src/components/sidebar/widgets/WidgetsBar.svelte
+++ b/plugins/workbench-resources/src/components/sidebar/widgets/WidgetsBar.svelte
@@ -33,7 +33,7 @@
     if (selected === widget._id) {
       minimizeSidebar(true)
     } else {
-      openWidget(widget, $sidebarStore.widgetsState.get(widget._id)?.data, true)
+      openWidget(widget, $sidebarStore.widgetsState.get(widget._id)?.data, { active: true, openedByUser: true })
     }
   }
 
diff --git a/plugins/workbench-resources/src/sidebar.ts b/plugins/workbench-resources/src/sidebar.ts
index 342feec535..b2323e858c 100644
--- a/plugins/workbench-resources/src/sidebar.ts
+++ b/plugins/workbench-resources/src/sidebar.ts
@@ -32,6 +32,7 @@ export interface WidgetState {
   tabs: WidgetTab[]
   tab?: string
   closedByUser?: boolean
+  openedByUser?: boolean
 }
 
 export interface SidebarState {
@@ -97,16 +98,23 @@ function setSidebarStateToLocalStorage (state: SidebarState): void {
   )
 }
 
-export function openWidget (widget: Widget, data?: Record<string, any>, active = true): void {
+export function openWidget (
+  widget: Widget,
+  data?: Record<string, any>,
+  params?: { active: boolean, openedByUser: boolean }
+): void {
   const state = get(sidebarStore)
   const { widgetsState } = state
   const widgetState = widgetsState.get(widget._id)
+  const active = params?.active ?? true
+  const openedByUser = params?.openedByUser ?? false
 
   widgetsState.set(widget._id, {
     _id: widget._id,
     data: data ?? widgetState?.data,
     tab: widgetState?.tab,
-    tabs: widgetState?.tabs ?? []
+    tabs: widgetState?.tabs ?? [],
+    openedByUser
   })
 
   Analytics.handleEvent('workbench.OpenSidebarWidget', { widget: widget._id })