Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Jan 12, 2026

Implements adjustable sidebar width with drag handle and localStorage persistence


View task on Roo Code Cloud


Important

Introduces adjustable sidebar width in ChatView using ResizablePanels, with drag handle and localStorage persistence.

  • Feature:
    • Introduces adjustable sidebar width in ChatView.tsx using ResizablePanels.
    • Sidebar width can be adjusted with a drag handle and persists using localStorage.
  • Components:
    • Adds ResizablePanels component in ResizablePanels.tsx.
    • ResizablePanels handles left and right panel widths, with defaults and constraints.
  • Behavior:
    • Sidebar width is adjustable between 20% and 65% of the total width.
    • Width changes are saved to localStorage under task-view-sidebar-width.

This description was created by Ellipsis for 8b37ea7. You can customize this summary. It will automatically update as commits are pushed.

@roomote
Copy link
Contributor Author

roomote bot commented Jan 12, 2026

Rooviewer Clock   See task on Roo Cloud

Review complete. Found 2 issues that need attention:

  • Duplicate TodoListDisplay - The todo list is rendered twice in the left panel (once in TaskHeader, once separately in ChatView)
  • Right panel layout - Missing flex flex-col on the right panel wrapper, causing the chat messages area to not fill available height

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jan 12, 2026
Comment on lines +1442 to +1447
{/* Add todo list to sidebar if available */}
{latestTodos && latestTodos.length > 0 && (
<div className="px-3 mt-3">
<TodoListDisplay todos={latestTodos} />
</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.

This TodoListDisplay duplicates the one already rendered inside TaskHeader (line 411 of TaskHeader.tsx). When hasTodos is true, TaskHeader shows the todo list at the bottom of the task card, so adding it here results in two identical todo lists in the left panel. Either remove this block or remove the todo display from TaskHeader if the intent is to show todos separately.

Fix it with Roo Code or mention @roomote and request a fix.

</div>

{/* Right Panel */}
<div style={{ width: `${rightWidth}%` }} className="overflow-hidden">
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 right panel wrapper needs flex flex-col for the grow class on the Virtuoso container (in ChatView.tsx) to work correctly. Without a flex parent, the chat messages area won't expand to fill the available height.

Suggested change
<div style={{ width: `${rightWidth}%` }} className="overflow-hidden">
<div style={{ width: `${rightWidth}%` }} className="overflow-hidden flex flex-col">

Fix it with Roo Code or mention @roomote and request a fix.

@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jan 12, 2026
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants