From 71c5ff88dbfb245a298065c02e2f8166cd6d0cf4 Mon Sep 17 00:00:00 2001
From: Jeremy Eder <jeder@redhat.com>
Date: Thu, 29 Jan 2026 00:17:28 -0500
Subject: [PATCH] UI Polish: Fix cursor hover, context overflow, and trailing
 slash handling

This commit addresses three UI/UX issues:

- Fix #514: Add cursor:pointer to repository expand/collapse chevron button
  to provide visual feedback on interactive element

- Fix #519: Fix Context panel overflow by changing overflow-visible to min-w-0
  to properly truncate long repository URLs and keep remove button visible

- Fix #518: Sanitize repository URLs by removing trailing slashes before
  submission to prevent validation failures

All changes are CSS/input handling improvements with no functional impact.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 .../components/accordions/repositories-accordion.tsx         | 4 ++--
 .../[sessionName]/components/modals/add-context-modal.tsx    | 5 ++++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/accordions/repositories-accordion.tsx b/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/accordions/repositories-accordion.tsx
index 38e8524ee..27cfc3f12 100644
--- a/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/accordions/repositories-accordion.tsx
+++ b/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/accordions/repositories-accordion.tsx
@@ -124,7 +124,7 @@ export function RepositoriesAccordion({
                       {hasBranches && (
                         <button
                           onClick={toggleExpanded}
-                          className="h-4 w-4 text-muted-foreground flex-shrink-0 hover:text-foreground"
+                          className="h-4 w-4 text-muted-foreground flex-shrink-0 hover:text-foreground cursor-pointer"
                         >
                           {isExpanded ? (
                             <ChevronDown className="h-4 w-4" />
@@ -135,7 +135,7 @@ export function RepositoriesAccordion({
                       )}
                       {!hasBranches && <div className="h-4 w-4 flex-shrink-0" />}
                       <GitBranch className="h-4 w-4 text-muted-foreground flex-shrink-0" />
-                      <div className="flex-1 overflow-visible">
+                      <div className="flex-1 min-w-0">
                         <div className="flex items-center gap-2 flex-wrap">
                           <div className="text-sm font-medium truncate">{repoName}</div>
                           {currentBranch && (
diff --git a/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/modals/add-context-modal.tsx b/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/modals/add-context-modal.tsx
index 9a97869e9..6c7885b40 100644
--- a/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/modals/add-context-modal.tsx
+++ b/components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/modals/add-context-modal.tsx
@@ -34,9 +34,12 @@ export function AddContextModal({
   const handleSubmit = async () => {
     if (!contextUrl.trim()) return;
 
+    // Trim URL and remove trailing slash
+    const sanitizedUrl = contextUrl.trim().replace(/\/+$/, '');
+
     // Use autoBranch from backend (single source of truth), or empty to let runner auto-generate
     const defaultBranch = autoBranch || '';
-    await onAddRepository(contextUrl.trim(), contextBranch.trim() || defaultBranch, autoPush);
+    await onAddRepository(sanitizedUrl, contextBranch.trim() || defaultBranch, autoPush);
 
     // Reset form
     setContextUrl("");