Skip to content

fix(DATAGO-123246): Nav bar recent chats scroll & layout improvements#1147

Open
JKaram wants to merge 3 commits intomainfrom
JKaram/DATAGO-123246/nav-bar
Open

fix(DATAGO-123246): Nav bar recent chats scroll & layout improvements#1147
JKaram wants to merge 3 commits intomainfrom
JKaram/DATAGO-123246/nav-bar

Conversation

@JKaram
Copy link
Contributor

@JKaram JKaram commented Mar 6, 2026

Summary

  • Recent chats scroll behavior: Split the expanded nav into two independent flex sections — nav items (fixed height) and recent chats (flexible, scrollable). When nav items with children expand, the recent chats section shrinks dynamically and becomes scrollable.
  • Increased max recent chats: Changed MAX_RECENT_CHATS from 6 to 15 to show more recent conversations.
  • New Chat button no longer highlighted by route: The New Chat button is now a pure action button that always opens a fresh session without active-state styling.
  • Bottom nav overlaps recent chats: Bottom nav items have a solid background and z-index so recent chats scroll behind them cleanly.
  • Subtle scrollbar: Added a scrollbar-subtle CSS class that hides the scrollbar by default and reveals it on hover.

Test plan

  • Expand the navigation sidebar and verify up to 15 recent chats are shown
  • Expand a nav item with children (e.g., a submenu) and confirm recent chats shrinks and shows a scrollbar on hover
  • Confirm at least 3 chat items remain visible when submenus are expanded
  • Verify the New Chat button is never highlighted/active-styled
  • Verify bottom nav items stay fixed and recent chats scroll behind them
  • Collapse and re-expand the sidebar — verify state is preserved
  • Run npm run build to check for build errors
CleanShot.2026-03-06.at.11.39.31.mp4

Comment on lines -228 to 230
<div className={iconWrapperStyles({ active: activeItem === "chats" })}>
<NewChatIcon className={iconStyles({ active: activeItem === "chats" })} />
<div className={iconWrapperStyles({ active: false })}>
<NewChatIcon className={iconStyles({ active: false })} />
</div>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The New Chat button should not be highlighted while on the /chat route`

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

✅ FOSSA Guard: Licensing (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (cca8fcfd72d6c78db2a733f412c9431d65882069) • 0 new, 8 total (8 in base)

Scan Report | View Details in FOSSA

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

✅ FOSSA Guard: Vulnerability (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (cca8fcfd72d6c78db2a733f412c9431d65882069) • 0 new, 11 total (11 in base)

Scan Report | View Details in FOSSA

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

WhiteSource Policy Violation Summary

✅︎ No Blocking Whitesource Policy Violations found in solaceai/solace-agent-mesh-ui-pr-1147!

@sonarqube-solacecloud
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant