From e997bf65079b4540ef9a662d6c9ccaf165ffc994 Mon Sep 17 00:00:00 2001
From: chanx <1243304602@qq.com>
Date: Fri, 29 Aug 2025 16:54:01 +0800
Subject: [PATCH] Fix: Optimized the style and functionality of multiple
 components #3221 (#9824)

### What problem does this PR solve?

Fix: Optimized the style and functionality of multiple components #3221

- Modified the SkeletonCard component, adding a className attribute and
adjusting the style
- Updated the RAGFlowSelect component, adding a disabled attribute
- Adjusted the style of the Tooltip component
- Optimized the layout of the RetrievalTesting and TestingResult pages
- Updated the style and loading status display of NextSearch-related
pages
- Removed unnecessary logs from the Spotlight component

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
---
 web/src/components/skeleton-card.tsx          | 17 ++++++-----
 web/src/components/ui/select.tsx              |  8 ++++-
 web/src/components/ui/tooltip.tsx             |  4 +--
 web/src/pages/dataset/testing/index.tsx       | 12 ++++----
 .../pages/dataset/testing/testing-result.tsx  |  2 +-
 web/src/pages/next-search/search-view.tsx     | 29 ++++++++-----------
 web/src/pages/next-search/spotlight.tsx       |  1 -
 7 files changed, 39 insertions(+), 34 deletions(-)

diff --git a/web/src/components/skeleton-card.tsx b/web/src/components/skeleton-card.tsx
index 76142c48e..aa565facb 100644
--- a/web/src/components/skeleton-card.tsx
+++ b/web/src/components/skeleton-card.tsx
@@ -1,13 +1,16 @@
 import { Skeleton } from '@/components/ui/skeleton';
+import { cn } from '@/lib/utils';
 
-export function SkeletonCard() {
+type SkeletonCardProps = {
+  className?: string;
+};
+export function SkeletonCard(props: SkeletonCardProps) {
+  const { className } = props;
   return (
-    <div className="flex flex-col space-y-3 items-center">
-      <Skeleton className="h-[125px] w-[250px] rounded-xl" />
-      <div className="space-y-2 w-[250px]">
-        <Skeleton className="h-4 w-[250px]" />
-        <Skeleton className="h-4 w-[200px]" />
-      </div>
+    <div className={cn('space-y-2', className)}>
+      <Skeleton className="h-4 w-full bg-bg-card" />
+      <Skeleton className="h-4 w-full bg-bg-card" />
+      <Skeleton className="h-4 w-2/3 bg-bg-card" />
     </div>
   );
 }
diff --git a/web/src/components/ui/select.tsx b/web/src/components/ui/select.tsx
index 3f030cbbb..acf39d6a8 100644
--- a/web/src/components/ui/select.tsx
+++ b/web/src/components/ui/select.tsx
@@ -224,6 +224,7 @@ export const RAGFlowSelect = forwardRef<
     allowClear,
     placeholder,
     contentProps = {},
+    disabled = false,
     // defaultValue,
     triggerClassName,
     onlyShowSelectedIcon = false,
@@ -278,7 +279,12 @@ export const RAGFlowSelect = forwardRef<
   }, [onlyShowSelectedIcon, options, value]);
 
   return (
-    <Select onValueChange={handleChange} value={value} key={key}>
+    <Select
+      onValueChange={handleChange}
+      value={value}
+      key={key}
+      disabled={disabled}
+    >
       <FormControlWidget>
         <SelectTrigger
           value={value}
diff --git a/web/src/components/ui/tooltip.tsx b/web/src/components/ui/tooltip.tsx
index 920bb4fb8..15bbc481c 100644
--- a/web/src/components/ui/tooltip.tsx
+++ b/web/src/components/ui/tooltip.tsx
@@ -107,7 +107,7 @@ export const AntToolTip: React.FC<AntToolTipProps> = ({
       {visible && title && (
         <div
           className={cn(
-            'absolute z-50 px-2.5 py-1.5 text-xs text-white bg-gray-800 rounded-sm shadow-sm whitespace-nowrap',
+            'absolute z-50 px-2.5 py-2 text-xs text-text-primary bg-muted rounded-sm shadow-sm whitespace-wrap',
             getPlacementClasses(),
             className,
           )}
@@ -115,7 +115,7 @@ export const AntToolTip: React.FC<AntToolTipProps> = ({
           {title}
           <div
             className={cn(
-              'absolute w-2 h-2 bg-gray-800',
+              'absolute w-2 h-2  bg-muted ',
               placement === 'top' &&
                 'bottom-[-4px] left-1/2 transform -translate-x-1/2 rotate-45',
               placement === 'bottom' &&
diff --git a/web/src/pages/dataset/testing/index.tsx b/web/src/pages/dataset/testing/index.tsx
index 358b89b14..d2d0dce55 100644
--- a/web/src/pages/dataset/testing/index.tsx
+++ b/web/src/pages/dataset/testing/index.tsx
@@ -40,11 +40,13 @@ export default function RetrievalTesting() {
                 <Plus /> Add New Test
               </Button> */}
             </div>
-            <TestingForm
-              loading={loading}
-              setValues={setValues}
-              refetch={refetch}
-            ></TestingForm>
+            <div className="h-[calc(100vh-241px)] overflow-auto scrollbar-thin">
+              <TestingForm
+                loading={loading}
+                setValues={setValues}
+                refetch={refetch}
+              ></TestingForm>
+            </div>
           </div>
           <TestingResult
             data={data}
diff --git a/web/src/pages/dataset/testing/testing-result.tsx b/web/src/pages/dataset/testing/testing-result.tsx
index 6ee794a34..b0e984009 100644
--- a/web/src/pages/dataset/testing/testing-result.tsx
+++ b/web/src/pages/dataset/testing/testing-result.tsx
@@ -77,7 +77,7 @@ export function TestingResult({
           <FilterButton></FilterButton>
         </FilterPopover>
       </div>
-      <section className="flex flex-col gap-5 overflow-auto h-[76vh] mb-5">
+      <section className="flex flex-col gap-5 overflow-auto h-[calc(100vh-241px)] scrollbar-thin mb-5">
         {data.chunks?.map((x) => (
           <FormContainer key={x.chunk_id} className="px-5 py-2.5">
             <ChunkTitle item={x}></ChunkTitle>
diff --git a/web/src/pages/next-search/search-view.tsx b/web/src/pages/next-search/search-view.tsx
index 710ab3b4d..e96eac3c6 100644
--- a/web/src/pages/next-search/search-view.tsx
+++ b/web/src/pages/next-search/search-view.tsx
@@ -1,6 +1,7 @@
 import { FileIcon } from '@/components/icon-font';
 import { ImageWithPopover } from '@/components/image';
 import { Input } from '@/components/originui/input';
+import { SkeletonCard } from '@/components/skeleton-card';
 import { Button } from '@/components/ui/button';
 import {
   Popover,
@@ -8,7 +9,6 @@ import {
   PopoverTrigger,
 } from '@/components/ui/popover';
 import { RAGFlowPagination } from '@/components/ui/ragflow-pagination';
-import { Skeleton } from '@/components/ui/skeleton';
 import { IReference } from '@/interfaces/database/chat';
 import { cn } from '@/lib/utils';
 import DOMPurify from 'dompurify';
@@ -157,11 +157,7 @@ export default function SearchingView({
                   {t('search.AISummary')}
                 </div>
                 {isEmpty(answer) && sendingLoading ? (
-                  <div className="space-y-2 mt-2">
-                    <Skeleton className="h-4 w-full bg-bg-card" />
-                    <Skeleton className="h-4 w-full bg-bg-card" />
-                    <Skeleton className="h-4 w-2/3 bg-bg-card" />
-                  </div>
+                  <SkeletonCard className=" mt-2" />
                 ) : (
                   answer.answer && (
                     <div className="border rounded-lg p-4 mt-3 max-h-52 overflow-auto scrollbar-none">
@@ -195,12 +191,9 @@ export default function SearchingView({
                 <>
                   {chunks.map((chunk, index) => {
                     return (
-                      <>
-                        <div
-                          key={chunk.chunk_id}
-                          className="w-full flex flex-col"
-                        >
-                          <div className="w-full">
+                      <div key={index}>
+                        <div className="w-full flex flex-col">
+                          <div className="w-full highlightContent">
                             <ImageWithPopover
                               id={chunk.img_id}
                             ></ImageWithPopover>
@@ -215,10 +208,12 @@ export default function SearchingView({
                                   className="text-sm text-text-primary mb-1"
                                 ></div>
                               </PopoverTrigger>
-                              <PopoverContent className="text-text-primary">
-                                <HightLightMarkdown>
-                                  {chunk.content_with_weight}
-                                </HightLightMarkdown>
+                              <PopoverContent className="text-text-primary !w-full max-w-lg ">
+                                <div className="max-h-96 overflow-auto scrollbar-thin">
+                                  <HightLightMarkdown>
+                                    {chunk.content_with_weight}
+                                  </HightLightMarkdown>
+                                </div>
                               </PopoverContent>
                             </Popover>
                           </div>
@@ -235,7 +230,7 @@ export default function SearchingView({
                         {index < chunks.length - 1 && (
                           <div className="w-full border-b border-border-default/80 mt-6"></div>
                         )}
-                      </>
+                      </div>
                     );
                   })}
                 </>
diff --git a/web/src/pages/next-search/spotlight.tsx b/web/src/pages/next-search/spotlight.tsx
index 2e1e3dc85..3d9b31f4a 100644
--- a/web/src/pages/next-search/spotlight.tsx
+++ b/web/src/pages/next-search/spotlight.tsx
@@ -7,7 +7,6 @@ interface SpotlightProps {
 
 const Spotlight: React.FC<SpotlightProps> = ({ className }) => {
   const isDark = useIsDarkTheme();
-  console.log('isDark', isDark);
   return (
     <div
       className={`absolute inset-0  opacity-80 ${className} rounded-lg`}