From a59bab4dc7ca9fd10898a98037e0f749154bc3eb Mon Sep 17 00:00:00 2001
From: Marco Beretta <81851188+berry-13@users.noreply.github.com>
Date: Mon, 29 Dec 2025 19:49:18 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=A0=20style:=20Expanded=20Thinking=20f?=
 =?UTF-8?q?ooter,=20Banner=20links,=20and=20Copy=20Thoughts=20accessibilit?=
 =?UTF-8?q?y=20(#11142)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* feat(Thinking): Add ThinkingFooter component for improved UX

* style(Banner): auto-style hyperlinks in banner messages

* fix: Simplify ThinkingFooter component by removing unused content prop and update aria-label for accessibility

* fix: Correct import order for consistency in Thinking component

* fix(ThinkingFooter): Update documentation to clarify footer functionality
---
 client/src/components/Banners/Banner.tsx      |  7 ++-
 .../Chat/Messages/Content/Parts/Reasoning.tsx |  5 +-
 .../Chat/Messages/Content/Parts/Thinking.tsx  | 49 +++++++++++++++++--
 client/src/locales/en/translation.json        |  1 +
 4 files changed, 53 insertions(+), 9 deletions(-)

diff --git a/client/src/components/Banners/Banner.tsx b/client/src/components/Banners/Banner.tsx
index 20b2f8c270..b10fa70941 100644
--- a/client/src/components/Banners/Banner.tsx
+++ b/client/src/components/Banners/Banner.tsx
@@ -38,10 +38,13 @@ export const Banner = ({ onHeightChange }: { onHeightChange?: (height: number) =
   return (
     <div
       ref={bannerRef}
-      className="sticky top-0 z-20 flex items-center bg-surface-secondary px-2 py-1 text-text-primary dark:bg-gradient-to-r md:relative"
+      className="sticky top-0 z-20 flex items-center bg-presentation px-2 py-1 text-text-primary dark:bg-gradient-to-r md:relative"
     >
       <div
-        className={cn('text-md w-full truncate text-center', !banner.persistable && 'px-4')}
+        className={cn(
+          'text-md w-full truncate text-center [&_a]:text-blue-700 [&_a]:underline dark:[&_a]:text-blue-400',
+          !banner.persistable && 'px-4',
+        )}
         dangerouslySetInnerHTML={{ __html: banner.message }}
       ></div>
       {!banner.persistable && (
diff --git a/client/src/components/Chat/Messages/Content/Parts/Reasoning.tsx b/client/src/components/Chat/Messages/Content/Parts/Reasoning.tsx
index 29f2e7187a..b08619e4c8 100644
--- a/client/src/components/Chat/Messages/Content/Parts/Reasoning.tsx
+++ b/client/src/components/Chat/Messages/Content/Parts/Reasoning.tsx
@@ -2,7 +2,7 @@ import { memo, useMemo, useState, useCallback } from 'react';
 import { useAtom } from 'jotai';
 import type { MouseEvent } from 'react';
 import { ContentTypes } from 'librechat-data-provider';
-import { ThinkingContent, ThinkingButton } from './Thinking';
+import { ThinkingContent, ThinkingButton, ThinkingFooter } from './Thinking';
 import { showThinkingAtom } from '~/store/showThinking';
 import { useMessageContext } from '~/Providers';
 import { useLocalize } from '~/hooks';
@@ -69,7 +69,7 @@ const Reasoning = memo(({ reasoning, isLast }: ReasoningProps) => {
   return (
     <div className="group/reasoning">
       <div className="group/thinking-container">
-        <div className="sticky top-0 z-10 mb-2 bg-presentation pb-2 pt-2">
+        <div className="mb-2 pb-2 pt-2">
           <ThinkingButton
             isExpanded={isExpanded}
             onClick={handleClick}
@@ -88,6 +88,7 @@ const Reasoning = memo(({ reasoning, isLast }: ReasoningProps) => {
         >
           <div className="overflow-hidden">
             <ThinkingContent>{reasoningText}</ThinkingContent>
+            <ThinkingFooter onClick={handleClick} />
           </div>
         </div>
       </div>
diff --git a/client/src/components/Chat/Messages/Content/Parts/Thinking.tsx b/client/src/components/Chat/Messages/Content/Parts/Thinking.tsx
index 71dbcd6af0..b26997171e 100644
--- a/client/src/components/Chat/Messages/Content/Parts/Thinking.tsx
+++ b/client/src/components/Chat/Messages/Content/Parts/Thinking.tsx
@@ -1,7 +1,7 @@
 import { useState, useMemo, memo, useCallback } from 'react';
 import { useAtomValue } from 'jotai';
-import { Lightbulb, ChevronDown } from 'lucide-react';
 import { Clipboard, CheckMark } from '@librechat/client';
+import { Lightbulb, ChevronDown, ChevronUp } from 'lucide-react';
 import type { MouseEvent, FC } from 'react';
 import { showThinkingAtom } from '~/store/showThinking';
 import { fontSizeAtom } from '~/store/fontSize';
@@ -90,13 +90,13 @@ export const ThinkingButton = memo(
           <button
             type="button"
             onClick={handleCopy}
-            title={
+            aria-label={
               isCopied
                 ? localize('com_ui_copied_to_clipboard')
                 : localize('com_ui_copy_thoughts_to_clipboard')
             }
             className={cn(
-              'rounded-lg p-1.5 text-text-secondary-alt transition-colors duration-200',
+              'rounded-lg p-1.5 text-text-secondary-alt',
               isExpanded
                 ? 'opacity-0 group-focus-within/thinking-container:opacity-100 group-hover/thinking-container:opacity-100'
                 : 'opacity-0',
@@ -104,7 +104,16 @@ export const ThinkingButton = memo(
               'focus-visible:opacity-100 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-black dark:focus-visible:ring-white',
             )}
           >
-            {isCopied ? <CheckMark className="h-[18px] w-[18px]" /> : <Clipboard size="19" />}
+            <span className="sr-only">
+              {isCopied
+                ? localize('com_ui_copied_to_clipboard')
+                : localize('com_ui_copy_thoughts_to_clipboard')}
+            </span>
+            {isCopied ? (
+              <CheckMark className="h-[18px] w-[18px]" aria-hidden="true" />
+            ) : (
+              <Clipboard size="19" aria-hidden="true" />
+            )}
           </button>
         )}
       </div>
@@ -112,6 +121,34 @@ export const ThinkingButton = memo(
   },
 );
 
+/**
+ * ThinkingFooter - Footer with collapse button shown at the bottom of expanded content
+ * Allows users to collapse without scrolling back to the top
+ */
+export const ThinkingFooter = memo(
+  ({ onClick }: { onClick: (e: MouseEvent<HTMLButtonElement>) => void }) => {
+    const localize = useLocalize();
+
+    return (
+      <div className="mt-3 flex items-center justify-end gap-2">
+        <button
+          type="button"
+          onClick={onClick}
+          aria-label={localize('com_ui_collapse_thoughts')}
+          className={cn(
+            'rounded-lg p-1.5 text-text-secondary-alt',
+            'hover:bg-surface-hover hover:text-text-primary',
+            'focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-black dark:focus-visible:ring-white',
+          )}
+        >
+          <span className="sr-only">{localize('com_ui_collapse_thoughts')}</span>
+          <ChevronUp className="h-[18px] w-[18px]" aria-hidden="true" />
+        </button>
+      </div>
+    );
+  },
+);
+
 /**
  * Thinking Component (LEGACY SYSTEM)
  *
@@ -153,7 +190,7 @@ const Thinking: React.ElementType = memo(({ children }: { children: React.ReactN
 
   return (
     <div className="group/thinking-container">
-      <div className="sticky top-0 z-10 mb-4 bg-presentation pb-2 pt-2">
+      <div className="mb-4 pb-2 pt-2">
         <ThinkingButton
           isExpanded={isExpanded}
           onClick={handleClick}
@@ -169,6 +206,7 @@ const Thinking: React.ElementType = memo(({ children }: { children: React.ReactN
       >
         <div className="overflow-hidden">
           <ThinkingContent>{children}</ThinkingContent>
+          <ThinkingFooter onClick={handleClick} />
         </div>
       </div>
     </div>
@@ -177,6 +215,7 @@ const Thinking: React.ElementType = memo(({ children }: { children: React.ReactN
 
 ThinkingButton.displayName = 'ThinkingButton';
 ThinkingContent.displayName = 'ThinkingContent';
+ThinkingFooter.displayName = 'ThinkingFooter';
 Thinking.displayName = 'Thinking';
 
 export default memo(Thinking);
diff --git a/client/src/locales/en/translation.json b/client/src/locales/en/translation.json
index 1361da7189..e833988d68 100644
--- a/client/src/locales/en/translation.json
+++ b/client/src/locales/en/translation.json
@@ -798,6 +798,7 @@
   "com_ui_close_window": "Close Window",
   "com_ui_code": "Code",
   "com_ui_collapse": "Collapse",
+  "com_ui_collapse_thoughts": "Collapse Thoughts",
   "com_ui_collapse_chat": "Collapse Chat",
   "com_ui_command_placeholder": "Optional: Enter a command for the prompt or name will be used",
   "com_ui_command_usage_placeholder": "Select a Prompt by command or name",