From e6512f07b64b185c38da8c7aa0acbe722d48e0ee Mon Sep 17 00:00:00 2001
From: Cole Goldsmith <cole.b.goldsmith@gmail.com>
Date: Wed, 12 Nov 2025 11:08:35 -0600
Subject: [PATCH] Settings page design sweep (#382)

* settings page design sweep

* Updated step list on onboarding upload

* fixed aria label

---------

Co-authored-by: Lucas Oliveira <lucas.edu.oli@hotmail.com>
---
 frontend/components/navigation.tsx            |  6 ++---
 .../components/onboarding-upload.tsx          |  2 +-
 .../settings/components/model-providers.tsx   |  5 ++--
 frontend/src/app/upload/[provider]/page.tsx   | 26 +++++++++----------
 frontend/src/components/chat-renderer.tsx     |  4 +--
 5 files changed, 21 insertions(+), 22 deletions(-)

diff --git a/frontend/components/navigation.tsx b/frontend/components/navigation.tsx
index 28422445..30999998 100644
--- a/frontend/components/navigation.tsx
+++ b/frontend/components/navigation.tsx
@@ -284,19 +284,19 @@ export function Navigation({
 			label: "Chat",
 			icon: MessageSquare,
 			href: "/chat",
-			active: pathname === "/" || pathname === "/chat",
+			active: pathname === "/" || pathname.startsWith("/chat"),
 		},
 		{
 			label: "Knowledge",
 			icon: Library,
 			href: "/knowledge",
-			active: pathname === "/knowledge",
+			active: pathname.startsWith("/knowledge"),
 		},
 		{
 			label: "Settings",
 			icon: Settings2,
 			href: "/settings",
-			active: pathname === "/settings",
+			active: pathname.startsWith("/settings"),
 		},
 	];
 
diff --git a/frontend/src/app/onboarding/components/onboarding-upload.tsx b/frontend/src/app/onboarding/components/onboarding-upload.tsx
index c12f3ca6..d9440e5a 100644
--- a/frontend/src/app/onboarding/components/onboarding-upload.tsx
+++ b/frontend/src/app/onboarding/components/onboarding-upload.tsx
@@ -15,7 +15,7 @@ const OnboardingUpload = ({ onComplete }: OnboardingUploadProps) => {
 	const [isUploading, setIsUploading] = useState(false);
 	const [currentStep, setCurrentStep] = useState<number | null>(null);
 
-	const STEP_LIST = ["Uploading your document", "Processing your document"];
+	const STEP_LIST = ["Uploading your document", "Generating embeddings", "Ingesting document", "Processing your document"];
 
 	// Query tasks to track completion
 	const { data: tasks } = useGetTasksQuery({
diff --git a/frontend/src/app/settings/components/model-providers.tsx b/frontend/src/app/settings/components/model-providers.tsx
index 4b78e792..96651230 100644
--- a/frontend/src/app/settings/components/model-providers.tsx
+++ b/frontend/src/app/settings/components/model-providers.tsx
@@ -105,7 +105,7 @@ export const ModelProviders = () => {
 
 	return (
 		<>
-			<div className="grid gap-6 md:grid-cols-2 lg:grid-cols-4">
+			<div className="grid gap-6 xs:grid-cols-1 md:grid-cols-2 lg:grid-cols-4">
 				{sortedProviderKeys.map((providerKey) => {
 					const {
 						name,
@@ -155,13 +155,14 @@ export const ModelProviders = () => {
 										<CardTitle className="flex flex-row items-center gap-2">
 											{name}
 											{isCurrentProvider && (
-												<div
+												<span
 													className={cn(
 														"h-2 w-2 rounded-full",
 														isProviderUnhealthy
 															? "bg-destructive"
 															: "bg-accent-emerald-foreground",
 													)}
+                          aria-label={isProviderUnhealthy ? "Error" : "Active"}
 												/>
 											)}
 										</CardTitle>
diff --git a/frontend/src/app/upload/[provider]/page.tsx b/frontend/src/app/upload/[provider]/page.tsx
index b144106d..1cad1b57 100644
--- a/frontend/src/app/upload/[provider]/page.tsx
+++ b/frontend/src/app/upload/[provider]/page.tsx
@@ -223,12 +223,10 @@ export default function UploadProviderPage() {
 
   if (isLoading) {
     return (
-      <div className="container mx-auto p-6">
-        <div className="flex items-center justify-center py-12">
-          <div className="text-center">
-            <div className="animate-spin rounded-full h-8 w-8 border-b-2 border-primary mx-auto mb-4"></div>
-            <p>Loading {getProviderDisplayName()} connector...</p>
-          </div>
+      <div className="flex items-center justify-center py-12">
+        <div className="text-center">
+          <div className="animate-spin rounded-full h-8 w-8 border-b-2 border-primary mx-auto mb-4"></div>
+          <p>Loading {getProviderDisplayName()} connector...</p>
         </div>
       </div>
     );
@@ -236,7 +234,7 @@ export default function UploadProviderPage() {
 
   if (error || !connector) {
     return (
-      <div className="container mx-auto p-6">
+      <>
         <div className="mb-6">
           <Button
             variant="ghost"
@@ -260,13 +258,13 @@ export default function UploadProviderPage() {
             </Button>
           </div>
         </div>
-      </div>
+      </>
     );
   }
 
   if (connector.status !== "connected") {
     return (
-      <div className="container mx-auto p-6">
+      <>
         <div className="mb-6">
           <Button
             variant="ghost"
@@ -293,13 +291,13 @@ export default function UploadProviderPage() {
             </Button>
           </div>
         </div>
-      </div>
+      </>
     );
   }
 
   if (!connector.hasAccessToken) {
     return (
-      <div className="container mx-auto p-6">
+      <>
         <div className="mb-6">
           <Button
             variant="ghost"
@@ -326,14 +324,14 @@ export default function UploadProviderPage() {
             </Button>
           </div>
         </div>
-      </div>
+      </>
     );
   }
 
   const hasSelectedFiles = selectedFiles.length > 0;
 
   return (
-    <div className="container mx-auto max-w-3xl px-6">
+    <>
       <div className="mb-8 flex gap-2 items-center">
         <Button variant="ghost" onClick={() => router.back()} size="icon">
           <ArrowLeft size={18} />
@@ -394,6 +392,6 @@ export default function UploadProviderPage() {
           </Tooltip>
         </div>
       </div>
-    </div>
+    </>
   );
 }
diff --git a/frontend/src/components/chat-renderer.tsx b/frontend/src/components/chat-renderer.tsx
index 53b9c282..c07471f5 100644
--- a/frontend/src/components/chat-renderer.tsx
+++ b/frontend/src/components/chat-renderer.tsx
@@ -95,8 +95,8 @@ export function ChatRenderer({
 	};
 
 	// List of paths with smaller max-width
-	const smallWidthPaths = ["/settings/connector/new"];
-	const isSmallWidthPath = smallWidthPaths.includes(pathname);
+	const smallWidthPaths = ["/settings", "/upload"];
+	const isSmallWidthPath = smallWidthPaths.some(path => pathname.startsWith(path));
 
 	const x = showLayout ? "0px" : `calc(-${SIDEBAR_WIDTH / 2}px + 50vw)`;
 	const y = showLayout ? "0px" : `calc(-${HEADER_HEIGHT / 2}px + 50vh)`;