From f50249e74feb2c6877a1f46d8856434577256bb3 Mon Sep 17 00:00:00 2001
From: josc146 <josStorer@outlook.com>
Date: Sun, 4 Aug 2024 15:15:48 +0800
Subject: [PATCH] improve b96ba7c0, getUserConfig() in realtime when mounting
 component

---
 src/content-script/index.jsx                  | 22 +++++++++++--------
 .../site-adapters/baidu/index.mjs             |  2 +-
 .../site-adapters/bilibili/index.mjs          |  2 +-
 .../site-adapters/github/index.mjs            |  4 ++--
 .../site-adapters/youtube/index.mjs           |  2 +-
 5 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/src/content-script/index.jsx b/src/content-script/index.jsx
index 713dd07b..7ef4eab7 100644
--- a/src/content-script/index.jsx
+++ b/src/content-script/index.jsx
@@ -31,9 +31,10 @@ import NotificationForChatGPTWeb from '../components/NotificationForChatGPTWeb'
 
 /**
  * @param {SiteConfig} siteConfig
- * @param {UserConfig} userConfig
  */
-async function mountComponent(siteConfig, userConfig) {
+async function mountComponent(siteConfig) {
+  const userConfig = await getUserConfig()
+
   if (!userConfig.alwaysFloatingSidebar) {
     const retry = 10
     let oldUrl = location.href
@@ -74,17 +75,19 @@ async function mountComponent(siteConfig, userConfig) {
     e.remove()
   })
 
-  const position = {
-    x: window.innerWidth - 300 - Math.floor((20 / 100) * window.innerWidth),
-    y: window.innerHeight / 2 - 200,
-  }
-  const toolbarContainer = createElementAtPosition(position.x, position.y)
-  toolbarContainer.className = 'chatgptbox-toolbar-container-not-queryable'
   if (userConfig.alwaysFloatingSidebar && question) {
+    const position = {
+      x: window.innerWidth - 300 - Math.floor((20 / 100) * window.innerWidth),
+      y: window.innerHeight / 2 - 200,
+    }
+    const toolbarContainer = createElementAtPosition(position.x, position.y)
+    toolbarContainer.className = 'chatgptbox-toolbar-container-not-queryable'
+
     let triggered = false
     if (userConfig.triggerMode === 'always') triggered = true
     else if (userConfig.triggerMode === 'questionMark' && endsWithQuestionMark(question.trim()))
       triggered = true
+
     render(
       <FloatingToolbar
         session={initSession({ modelName: userConfig.modelName })}
@@ -98,6 +101,7 @@ async function mountComponent(siteConfig, userConfig) {
     )
     return
   }
+
   const container = document.createElement('div')
   container.id = 'chatgptbox-container'
   render(
@@ -313,7 +317,7 @@ async function prepareForStaticCard() {
       }
     }
 
-    if (initSuccess) mountComponent(siteConfig[siteName], userConfig)
+    if (initSuccess) mountComponent(siteConfig[siteName])
   }
 }
 
diff --git a/src/content-script/site-adapters/baidu/index.mjs b/src/content-script/site-adapters/baidu/index.mjs
index 96ade831..2cae5447 100644
--- a/src/content-script/site-adapters/baidu/index.mjs
+++ b/src/content-script/site-adapters/baidu/index.mjs
@@ -14,7 +14,7 @@ export default {
         ) {
           const searchValue = await getInput(config.baidu.inputQuery)
           if (searchValue) {
-            mountComponent(config.baidu, userConfig)
+            mountComponent(config.baidu)
           }
         }
       })
diff --git a/src/content-script/site-adapters/bilibili/index.mjs b/src/content-script/site-adapters/bilibili/index.mjs
index 8e9f4080..bca96bd6 100644
--- a/src/content-script/site-adapters/bilibili/index.mjs
+++ b/src/content-script/site-adapters/bilibili/index.mjs
@@ -14,7 +14,7 @@ export default {
         const newPath = getVideoPath()
         if (newPath !== oldPath) {
           oldPath = newPath
-          mountComponent(config.bilibili, userConfig)
+          mountComponent(config.bilibili)
         }
       }
       window.setInterval(checkPathChange, 500)
diff --git a/src/content-script/site-adapters/github/index.mjs b/src/content-script/site-adapters/github/index.mjs
index d41a5695..da608271 100644
--- a/src/content-script/site-adapters/github/index.mjs
+++ b/src/content-script/site-adapters/github/index.mjs
@@ -124,13 +124,13 @@ export default {
         if (location.href !== oldUrl) {
           oldUrl = location.href
           if (isPull() || isIssue()) {
-            mountComponent(config.github, userConfig)
+            mountComponent(config.github)
             return
           }
 
           const patchUrl = await getPatchUrl()
           if (patchUrl) {
-            mountComponent(config.github, userConfig)
+            mountComponent(config.github)
           }
         }
       }
diff --git a/src/content-script/site-adapters/youtube/index.mjs b/src/content-script/site-adapters/youtube/index.mjs
index 233a1bbf..46f1a7fc 100644
--- a/src/content-script/site-adapters/youtube/index.mjs
+++ b/src/content-script/site-adapters/youtube/index.mjs
@@ -14,7 +14,7 @@ export default {
       const checkUrlChange = async () => {
         if (location.href !== oldUrl) {
           oldUrl = location.href
-          mountComponent(config.youtube, userConfig)
+          mountComponent(config.youtube)
         }
       }
       window.setInterval(checkUrlChange, 500)