From f76373c87fe57fd1976b3bfa5d0ce0688ea2326d Mon Sep 17 00:00:00 2001 From: Caroline McKenzie Date: Tue, 15 Jul 2025 15:42:54 -0400 Subject: [PATCH 1/3] restyle button with diff icon and padding --- .../components/sessions/SessionHistoryView.tsx | 15 ++++++++------- ui/desktop/src/components/ui/BackButton.tsx | 8 ++++---- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/ui/desktop/src/components/sessions/SessionHistoryView.tsx b/ui/desktop/src/components/sessions/SessionHistoryView.tsx index 23d764743d76..1fd7f00784e8 100644 --- a/ui/desktop/src/components/sessions/SessionHistoryView.tsx +++ b/ui/desktop/src/components/sessions/SessionHistoryView.tsx @@ -10,7 +10,6 @@ import { Target, LoaderCircle, AlertCircle, - ChevronLeft, ExternalLink, } from 'lucide-react'; import { type SessionDetails } from '../../sessions'; @@ -32,6 +31,7 @@ import ProgressiveMessageList from '../ProgressiveMessageList'; import { SearchView } from '../conversation/SearchView'; import { ChatContextManagerProvider } from '../context_management/ChatContextManager'; import { Message } from '../../types/message'; +import BackButton from '../ui/BackButton'; // Helper function to determine if a message is a user message (same as useChatEngine) const isUserMessage = (message: Message): boolean => { @@ -89,10 +89,11 @@ const SessionHeader: React.FC<{ return (
- +

{title}

{children}
@@ -372,12 +373,12 @@ const SessionHistoryView: React.FC = ({
-
+
{shareLink} ); From 73da7f84ca30c34964d5e5e7034e2ea19d188e40 Mon Sep 17 00:00:00 2001 From: Caroline McKenzie Date: Tue, 15 Jul 2025 15:57:08 -0400 Subject: [PATCH 2/3] spacing fixed on chat history page to accomodate --- ui/desktop/src/components/sessions/SessionHistoryView.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ui/desktop/src/components/sessions/SessionHistoryView.tsx b/ui/desktop/src/components/sessions/SessionHistoryView.tsx index 1fd7f00784e8..7442b831f581 100644 --- a/ui/desktop/src/components/sessions/SessionHistoryView.tsx +++ b/ui/desktop/src/components/sessions/SessionHistoryView.tsx @@ -88,14 +88,14 @@ const SessionHeader: React.FC<{ }> = ({ onBack, children, title, actionButtons }) => { return (
-
+
-

{title}

+

{title}

{children}
{actionButtons &&
{actionButtons}
}
From 740fb0fda5f542058b76a96ec978fd23e18b83ba Mon Sep 17 00:00:00 2001 From: Caroline McKenzie Date: Tue, 15 Jul 2025 16:26:34 -0400 Subject: [PATCH 3/3] unify the styling across the schedule and session history pages. --- .../src/components/schedule/ScheduleDetailView.tsx | 4 ++-- .../src/components/sessions/SessionHistoryView.tsx | 10 +++------- ui/desktop/src/components/ui/BackButton.tsx | 6 +++++- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/ui/desktop/src/components/schedule/ScheduleDetailView.tsx b/ui/desktop/src/components/schedule/ScheduleDetailView.tsx index 854ffe7bbd96..171f0ed61899 100644 --- a/ui/desktop/src/components/schedule/ScheduleDetailView.tsx +++ b/ui/desktop/src/components/schedule/ScheduleDetailView.tsx @@ -497,8 +497,8 @@ const ScheduleDetailView: React.FC = ({ scheduleId, onN
-

Schedule Details

-

Viewing Schedule ID: {scheduleId}

+

Schedule Details

+

Viewing Schedule ID: {scheduleId}

diff --git a/ui/desktop/src/components/sessions/SessionHistoryView.tsx b/ui/desktop/src/components/sessions/SessionHistoryView.tsx index 7442b831f581..ff5a6b8b21de 100644 --- a/ui/desktop/src/components/sessions/SessionHistoryView.tsx +++ b/ui/desktop/src/components/sessions/SessionHistoryView.tsx @@ -88,14 +88,10 @@ const SessionHeader: React.FC<{ }> = ({ onBack, children, title, actionButtons }) => { return (
-
- +
+
-

{title}

+

{title}

{children}
{actionButtons &&
{actionButtons}
}
diff --git a/ui/desktop/src/components/ui/BackButton.tsx b/ui/desktop/src/components/ui/BackButton.tsx index 12df7c4d305b..f927ff67ae53 100644 --- a/ui/desktop/src/components/ui/BackButton.tsx +++ b/ui/desktop/src/components/ui/BackButton.tsx @@ -3,6 +3,7 @@ import { ArrowLeft } from 'lucide-react'; import { Button } from './button'; import type { VariantProps } from 'class-variance-authority'; import { buttonVariants } from './button'; +import { cn } from '../../utils'; interface BackButtonProps extends VariantProps { onClick?: () => void; @@ -36,7 +37,10 @@ const BackButton: React.FC = ({ variant={variant} size={size} shape={shape} - className={className} + className={cn( + 'rounded-full px-6 py-2 flex items-center gap-2 text-text-default hover:cursor-pointer', + className + )} {...props} >