From f7abfbc31716ad21d35a9eaee58bb6cdca6ad57d Mon Sep 17 00:00:00 2001
From: Taylor Ho <taylorkmho@gmail.com>
Date: Tue, 8 Jul 2025 16:24:13 -0700
Subject: [PATCH] fix: better spacing around model options and summarize

---
 ui/desktop/src/components/ChatInput.tsx                  | 2 +-
 .../context_management/ManualSummaryButton.tsx           | 9 ++++++---
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/ui/desktop/src/components/ChatInput.tsx b/ui/desktop/src/components/ChatInput.tsx
index d94da5d715..86962ae688 100644
--- a/ui/desktop/src/components/ChatInput.tsx
+++ b/ui/desktop/src/components/ChatInput.tsx
@@ -978,7 +978,7 @@ export default function ChatInput({
               </>
             )}
             <ModelsBottomBar dropdownRef={dropdownRef} setView={setView} alerts={alerts} />
-            <div className="w-px h-4 bg-border-default mx-2"></div>
+            <div className="w-px h-4 bg-border-default mx-2" />
             <BottomMenuModeSelection setView={setView} />
             {messages.length > 0 && (
               <ManualSummarizeButton
diff --git a/ui/desktop/src/components/context_management/ManualSummaryButton.tsx b/ui/desktop/src/components/context_management/ManualSummaryButton.tsx
index 1b574f3726..1122f58338 100644
--- a/ui/desktop/src/components/context_management/ManualSummaryButton.tsx
+++ b/ui/desktop/src/components/context_management/ManualSummaryButton.tsx
@@ -1,5 +1,6 @@
 import React, { useState } from 'react';
 import { ScrollText } from 'lucide-react';
+import { cn } from '../../utils';
 import {
   Dialog,
   DialogContent,
@@ -47,11 +48,13 @@ export const ManualSummarizeButton: React.FC<ManualSummarizeButtonProps> = ({
 
   return (
     <>
+      <div className="w-px h-4 bg-border-default mx-2" />
       <div className="relative flex items-center">
         <button
-          className={`flex items-center justify-center text-text-default/70 hover:text-text-default text-xs ${
-            isLoadingSummary || isLoading ? 'cursor-not-allowed' : ''
-          }`}
+          className={cn(
+            'flex items-center justify-center text-text-default/70 hover:text-text-default text-xs cursor-pointer',
+            (isLoadingSummary || isLoading) && 'cursor-not-allowed'
+          )}
           onClick={handleClick}
           disabled={isLoadingSummary || isLoading}
           title="Summarize conversation context"