Skip to content

feat(lungo): add vertically resizable agent chat panel - #715

Merged
misi-bp merged 3 commits into
mainfrom
653/expandable-chat-area
Jul 27, 2026
Merged

feat(lungo): add vertically resizable agent chat panel#715
misi-bp merged 3 commits into
mainfrom
653/expandable-chat-area

Conversation

@misi-bp

@misi-bp misi-bp commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

This change improves the lungo main layout so the agent chat can be resized vertically against the workflow graph, similar to the existing horizontal sidebar split.

Behavior

  • When the user has sent a message, the main column uses a vertical react-resizable-panels group (graph above, chat below) with a drag handle (ChatPanelSeparator).
  • The chat panel starts sized to its content via useChatPanelContentSize, then switches to fill-height so dragging the separator grows or shrinks the scrollable message region.
  • Min/max constraints live in chatPanelLayout.ts (e.g. minimum chat height, graph keeps at least ~25% of the column).
  • The chat header minimize/maximize control is removed; expanding the conversation is done by dragging the separator instead of toggling collapse in the header.

Refactor

  • Shared ResizablePanelSeparator (and resizablePanelLayout constants) extracted from the sidebar separator and reused for both sidebar and chat dividers.

Scope: lungo frontend only (RootPage, ChatArea, layout components/hooks). No API or backend changes.

Issue Link

Fixes #653

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

Manual test plan

  1. Open lungo with no messages — chat stays composer-only; no vertical separator.
  2. Send a prompt — separator appears; drag up/down to resize graph vs chat.
  3. Confirm message thread scrolls when chat is tall and content overflows.
  4. Clear conversation — vertical chat panel and separator disappear; graph uses full height.
  5. Sidebar horizontal resize still works (shared separator component).

@misi-bp
misi-bp requested a review from a team as a code owner July 24, 2026 14:25
@misi-bp
misi-bp force-pushed the 653/expandable-chat-area branch from 42a1a90 to 24576e9 Compare July 24, 2026 14:41
@misi-bp

misi-bp commented Jul 25, 2026

Copy link
Copy Markdown
Contributor Author

This will be updated (rebased) after #712 is merged, conflicts expected.

Signed-off-by: misi-bp <mkaloczy@cisco.com>

@pregnor pregnor 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, I will approve after the merge conflict resolution

misi-bp added 2 commits July 27, 2026 16:04
Signed-off-by: misi-bp <mkaloczy@cisco.com>
@misi-bp
misi-bp force-pushed the 653/expandable-chat-area branch from 24576e9 to 7304ed4 Compare July 27, 2026 14:32

@delthazor delthazor 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

@misi-bp
misi-bp merged commit 8548a7b into main Jul 27, 2026
20 checks passed
@misi-bp
misi-bp deleted the 653/expandable-chat-area branch July 27, 2026 14:57
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.

[Feature]: Implement UI improvements after review on 6/18

3 participants