From 3cbd07b68dbfef779d1158c4049de35f836bfeed Mon Sep 17 00:00:00 2001 From: keval shah Date: Wed, 13 May 2026 15:30:07 -0400 Subject: [PATCH] fix(ui): distinguish Memory and Model icons (BrainCog / BrainCircuit) (#13111) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Memory base and Model Providers both rendered the plain "Brain" Lucide icon, which QA flagged as confusing — the two sections felt interchangeable at a glance. Repoint each surface to its own variant per product/manager direction: Model Providers / model trigger / settings nav: Brain → BrainCircuit Memory base (sidebar nav, modal, details header, empty states): Brain → BrainCog BrainCircuit was already in use for Language Models / LLM Operations in the component library, so the Model Providers surface now lines up with that family. Memory keeps its own distinct glyph (cog) to read as "managed knowledge" rather than "a model". Test fixture in sidebarSegmentedNav.test.tsx updated. Provider-list fixtures that happened to default to "Brain" for unrelated test data are left as-is. --- .../modelInputComponent/components/ModelTrigger.tsx | 2 +- src/frontend/src/modals/createMemoryModal/index.tsx | 2 +- .../MemoriesMainContent/components/MemoriesSidebar.tsx | 4 ++-- .../MemoriesMainContent/components/MemoryDetailsHeader.tsx | 5 ++++- .../MemoriesMainContent/components/NoMemorySelected.tsx | 2 +- .../components/__tests__/sidebarSegmentedNav.test.tsx | 2 +- .../flowSidebarComponent/components/sidebarSegmentedNav.tsx | 2 +- src/frontend/src/pages/SettingsPage/index.tsx | 2 +- .../pages/SettingsPage/pages/ModelProvidersPage/index.tsx | 2 +- 9 files changed, 13 insertions(+), 10 deletions(-) diff --git a/src/frontend/src/components/core/parameterRenderComponent/components/modelInputComponent/components/ModelTrigger.tsx b/src/frontend/src/components/core/parameterRenderComponent/components/modelInputComponent/components/ModelTrigger.tsx index a87d4af31d..f69df71270 100644 --- a/src/frontend/src/components/core/parameterRenderComponent/components/modelInputComponent/components/ModelTrigger.tsx +++ b/src/frontend/src/components/core/parameterRenderComponent/components/modelInputComponent/components/ModelTrigger.tsx @@ -57,7 +57,7 @@ const ModelTrigger = ({ onClick={onOpenManageProviders} >
diff --git a/src/frontend/src/modals/createMemoryModal/index.tsx b/src/frontend/src/modals/createMemoryModal/index.tsx index 32d926a210..e1f72aff7b 100644 --- a/src/frontend/src/modals/createMemoryModal/index.tsx +++ b/src/frontend/src/modals/createMemoryModal/index.tsx @@ -57,7 +57,7 @@ export default function CreateMemoryModal({ onSubmit={handleSubmit} > - + Create Memory diff --git a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoriesSidebar.tsx b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoriesSidebar.tsx index d030216dc3..4630664662 100644 --- a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoriesSidebar.tsx +++ b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoriesSidebar.tsx @@ -34,7 +34,7 @@ export function MemoriesSidebar({

Memories

@@ -69,7 +69,7 @@ export function MemoriesSidebar({ {!filteredMemories.length ? (

diff --git a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoryDetailsHeader.tsx b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoryDetailsHeader.tsx index b291f6280e..2c1f8f698f 100644 --- a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoryDetailsHeader.tsx +++ b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/MemoryDetailsHeader.tsx @@ -39,7 +39,10 @@ export function MemoryDetailsHeader({ return (

- +

{memory.name}

{memory.description && ( diff --git a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/NoMemorySelected.tsx b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/NoMemorySelected.tsx index eb8681f9f4..d883beacd0 100644 --- a/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/NoMemorySelected.tsx +++ b/src/frontend/src/pages/FlowPage/components/MemoriesMainContent/components/NoMemorySelected.tsx @@ -11,7 +11,7 @@ export const NoMemorySelected = () => ( > diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/__tests__/sidebarSegmentedNav.test.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/__tests__/sidebarSegmentedNav.test.tsx index 81f6d2c8da..06ade93177 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/__tests__/sidebarSegmentedNav.test.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/__tests__/sidebarSegmentedNav.test.tsx @@ -422,7 +422,7 @@ describe("SidebarSegmentedNav", () => { }); expect(NAV_ITEMS[6]).toEqual({ id: "memories", - icon: "Brain", + icon: "BrainCog", label: "Memories", tooltip: "Memories", }); diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarSegmentedNav.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarSegmentedNav.tsx index 42820abb09..abefb238f9 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarSegmentedNav.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarSegmentedNav.tsx @@ -60,7 +60,7 @@ export const NAV_ITEMS: NavItem[] = [ }, { id: "memories", - icon: "Brain", + icon: "BrainCog", label: "Memories", tooltip: "Memories", }, diff --git a/src/frontend/src/pages/SettingsPage/index.tsx b/src/frontend/src/pages/SettingsPage/index.tsx index 3892862c28..d81e0af2d4 100644 --- a/src/frontend/src/pages/SettingsPage/index.tsx +++ b/src/frontend/src/pages/SettingsPage/index.tsx @@ -75,7 +75,7 @@ export default function SettingsPage(): JSX.Element { href: "/settings/model-providers", icon: ( ), diff --git a/src/frontend/src/pages/SettingsPage/pages/ModelProvidersPage/index.tsx b/src/frontend/src/pages/SettingsPage/pages/ModelProvidersPage/index.tsx index 7a74e0bcea..27150e134d 100644 --- a/src/frontend/src/pages/SettingsPage/pages/ModelProvidersPage/index.tsx +++ b/src/frontend/src/pages/SettingsPage/pages/ModelProvidersPage/index.tsx @@ -14,7 +14,7 @@ export default function ModelProvidersPage() { > {t("modelProviders.pageTitle")}