Skip to content

Fix collapsed sidebar overlapping the chat history icon in Agentic UI - #4280

Merged
sejas merged 9 commits into
trunkfrom
stu-2071-collapsed-sidebar-overlaps-with-chat-history-icon
Jul 23, 2026
Merged

Fix collapsed sidebar overlapping the chat history icon in Agentic UI#4280
sejas merged 9 commits into
trunkfrom
stu-2071-collapsed-sidebar-overlaps-with-chat-history-icon

Conversation

@sejas

@sejas sejas commented Jul 21, 2026

Copy link
Copy Markdown
Member

Related issues

When the sidebar is collapsed, the floating "Show sidebar" toggle sits in the bottom-left corner of the main area — directly on top of the chat history icon, making both hard to use.

How AI was used in this PR

AI assisted with the implementation; the change was reviewed and verified manually.

Proposed Changes

  • Move the toggle sidebar visibility to the bottom and always visible.
  • Bring back the right toggle for preview site pane
    One alternative could be keep the toggle button always under the text box.

Testing Instructions

  • Open the Agentic UI with a session that shows the footer controls (chat history + new chat) at the bottom of the panel.
  • Collapse the sidebar.
  • Expand the sidebar and confirm the footer controls return to their original position.

Before

toggle-sidebar-button-before.mp4

After

toggle-sidebar-button-after.mp4

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@sejas sejas self-assigned this Jul 21, 2026
sejas and others added 4 commits July 23, 2026 11:18
…llapsed-sidebar-overlaps-with-chat-history-icon

# Conflicts:
#	apps/ui/src/components/user-menu/index.tsx
#	apps/ui/src/components/user-menu/style.module.css
…aps-with-chat-history-icon' into stu-2071-collapsed-sidebar-overlaps-with-chat-history-icon
@sejas
sejas marked this pull request as ready for review July 23, 2026 10:34
@sejas
sejas requested review from bcotrim and nightnei July 23, 2026 10:37
@sejas
sejas requested a review from shaunandrews July 23, 2026 10:45
@wpmobilebot

wpmobilebot commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing b7087fe vs trunk

app-size

Metric trunk b7087fe Diff Change
App Size (Mac) 1362.04 MB 1362.03 MB 0.01 MB ⚪ 0.0%

site-editor

Metric trunk b7087fe Diff Change
load 1066 ms 1071 ms +5 ms ⚪ 0.0%

site-startup

Metric trunk b7087fe Diff Change
siteCreation 6474 ms 6966 ms +492 ms 🔴 7.6%
siteStartup 2389 ms 2398 ms +9 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@youknowriad

Copy link
Copy Markdown
Contributor

Why not keep the toggle in the top?

@youknowriad

Copy link
Copy Markdown
Contributor

For me this feels a bit more hacky, as for the user it's going to mix something that is very global (sidebar toggle) with something very specific a button related to the prompt/chat.

So I think the top position is a more natural position. Yes, it does mix it with the site dropdown but that's more natural, at least for me.

@bcotrim bcotrim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

On Off
Image Image
@youknowriad

Copy link
Copy Markdown
Contributor

If we keep it in the bottom, at least let's remove the space at the left of the site dropdown (that space was probably meant to host the toggle button)

@sejas

sejas commented Jul 23, 2026

Copy link
Copy Markdown
Member Author

Let's go with Riad's PR #4309 , we can always go back to this PR if we think is necessary.

@sejas sejas closed this Jul 23, 2026
@sejas sejas reopened this Jul 23, 2026
@sejas

sejas commented Jul 23, 2026

Copy link
Copy Markdown
Member Author

I added the right preview site toggle to the composer.

Logged in Logged out
Screenshot 2026-07-23 at 15 33 30 Screenshot 2026-07-23 at 15 34 31
@sejas
sejas merged commit 2b22f20 into trunk Jul 23, 2026
14 checks passed
@sejas
sejas deleted the stu-2071-collapsed-sidebar-overlaps-with-chat-history-icon branch July 23, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants