From e2e8ded2c02f28909b6a384acd15cd21c28bf277 Mon Sep 17 00:00:00 2001
From: one <wangan.cs@gmail.com>
Date: Sat, 16 Aug 2025 23:20:38 +0800
Subject: [PATCH] refactor: improve style for ManageModelList and Tooltips
 (#9227)

* refactor(ManageModelsPopup): remove margin of Empty

* chore: use destroyOnHidden rather than deprecated destroyTooltipOnHide

* refactor: center Empty
---
 .../src/components/Popups/ApiKeyListPopup/item.tsx       | 2 +-
 src/renderer/src/pages/home/Messages/ChatFlowHistory.tsx | 2 +-
 src/renderer/src/pages/home/Messages/MessageGroup.tsx    | 2 +-
 .../ProviderSettings/ModelList/ManageModelsList.tsx      | 2 +-
 .../ProviderSettings/ModelList/ManageModelsPopup.tsx     | 9 ++++++++-
 5 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/renderer/src/components/Popups/ApiKeyListPopup/item.tsx b/src/renderer/src/components/Popups/ApiKeyListPopup/item.tsx
index 8150ee9b0c..75d32b8bc5 100644
--- a/src/renderer/src/components/Popups/ApiKeyListPopup/item.tsx
+++ b/src/renderer/src/components/Popups/ApiKeyListPopup/item.tsx
@@ -130,7 +130,7 @@ const ApiKeyItem: FC<ApiKeyItemProps> = ({
             mouseEnterDelay={0.5}
             placement="top"
             // 确保不留下明文
-            destroyTooltipOnHide>
+            destroyOnHidden>
             <span style={{ cursor: 'help' }}>{maskApiKey(keyStatus.key)}</span>
           </Tooltip>
 
diff --git a/src/renderer/src/pages/home/Messages/ChatFlowHistory.tsx b/src/renderer/src/pages/home/Messages/ChatFlowHistory.tsx
index 25801bdeba..4197b0a957 100644
--- a/src/renderer/src/pages/home/Messages/ChatFlowHistory.tsx
+++ b/src/renderer/src/pages/home/Messages/ChatFlowHistory.tsx
@@ -142,7 +142,7 @@ const CustomNode: FC<{ data: any }> = ({ data }) => {
       color="rgba(0, 0, 0, 0.85)"
       mouseEnterDelay={0.3}
       mouseLeaveDelay={0.1}
-      destroyTooltipOnHide>
+      destroyOnHidden>
       <CustomNodeContainer
         style={{
           borderColor,
diff --git a/src/renderer/src/pages/home/Messages/MessageGroup.tsx b/src/renderer/src/pages/home/Messages/MessageGroup.tsx
index 34c9c014ac..4db2a42747 100644
--- a/src/renderer/src/pages/home/Messages/MessageGroup.tsx
+++ b/src/renderer/src/pages/home/Messages/MessageGroup.tsx
@@ -241,7 +241,7 @@ const MessageGroup = ({ messages, topic, registerMessageElement }: Props) => {
         return (
           <Popover
             key={message.id}
-            destroyTooltipOnHide
+            destroyOnHidden
             content={
               <MessageWrapper
                 className={classNames([
diff --git a/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsList.tsx b/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsList.tsx
index a6d7020511..cd906a31c7 100644
--- a/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsList.tsx
+++ b/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsList.tsx
@@ -109,7 +109,7 @@ const ManageModelsList: React.FC<ManageModelsListProps> = ({ modelGroups, provid
 
       return (
         <Tooltip
-          destroyTooltipOnHide
+          destroyOnHidden
           title={
             isAllInProvider
               ? t('settings.models.manage.remove_whole_group')
diff --git a/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsPopup.tsx b/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsPopup.tsx
index ee4e87daba..6f32841434 100644
--- a/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsPopup.tsx
+++ b/src/renderer/src/pages/settings/ProviderSettings/ModelList/ManageModelsPopup.tsx
@@ -336,7 +336,14 @@ const PopupContainer: React.FC<Props> = ({ providerId, resolve }) => {
             <Empty
               image={Empty.PRESENTED_IMAGE_SIMPLE}
               description={t('settings.models.empty')}
-              style={{ visibility: loadingModels ? 'hidden' : 'visible' }}
+              style={{
+                visibility: loadingModels ? 'hidden' : 'visible',
+                display: 'flex',
+                justifyContent: 'center',
+                alignItems: 'center',
+                height: '100%',
+                margin: '0'
+              }}
             />
           ) : (
             <ManageModelsList