Skip to content

Conversation

@brunobergher
Copy link
Collaborator

@brunobergher brunobergher commented Nov 7, 2025

This is strictly UX-focused, with the aim of reducing visual noise and making to-do lists more useful. Changes include:

  • Redesigned to-do list component, with fewer elements and a strong emphasis on the task at hand, now part of the task header (so you don't need to close it in order to continue following Roo's work)
  • Because it has a more fixed place, I changed the to-do entries in the chat messages to only show the changed to-dos (usually the one just completed and the one Roo started on)
  • While integrating it to the task header, I made it visually lighter ("task" -> icon, chevron down only shows on hover, etc)
  • Deprioritized context-related info in the expanded state of the TaskHeader, moving the buttons closer to the task itself
  • And seceral of other visual tweaks at places

No functionality change (other than the shorter to-do-related messages in the chat.

Test Procedure

  1. Start a task where the agent comes up with a to-do list plan
  2. Watch to-dos fly by, interact open/close the task header

Screenshots / Videos

There many states, but these screenshots encapsulate the vast majority of what changes.



Important

Improves UX by redesigning to-do lists and task headers, integrating them for better focus and reduced visual noise, with various visual tweaks across the UI.

  • UX Improvements:
    • Redesigned to-do list component, integrated into task header for better focus and reduced visual noise.
    • Task header now shows only changed to-dos in chat messages, emphasizing current tasks.
    • Visual tweaks include lighter task header, hover effects, and deprioritized context info.
  • Components:
    • Added TodoChangeDisplay in ChatRow.tsx to show changes in to-dos.
    • Introduced LucideIconButton in LucideIconButton.tsx for consistent icon button styling.
    • Updated TaskHeader.tsx to integrate to-do list and adjust task info layout.
  • Miscellaneous:
    • Updated translations in multiple language files for new to-do list messages.
    • Adjusted CSS in index.css for mention highlights and other UI elements.

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

@roomote
Copy link
Contributor

roomote bot commented Nov 7, 2025

Rooviewer Clock   See task on Roo Cloud

Review completed. Latest commit 1334429 removes a commented-out line referencing an unused parameter. All previously identified issues remain resolved. No new issues found.

  • Fix invalid CSS class text-shrink-0 in TodoListDisplay.tsx (should be shrink-0)
  • Fix invalid Tailwind classes -mt-1.25 and max-w-50 in TaskHeader.tsx (use valid Tailwind utilities)
  • Remove unused CSS animation gradient-text in index.css
  • Remove unused prop isParentExpanded from TodoListDisplay component
Previous reviews

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 Nov 7, 2025
brunobergher and others added 6 commits November 7, 2025 12:21
Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
@brunobergher brunobergher marked this pull request as ready for review November 7, 2025 12:41
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. UI/UX UI/UX related or focused labels Nov 7, 2025
@roomote
Copy link
Contributor

roomote bot commented Nov 7, 2025

Rooviewer Clock   See task on Roo Cloud

Review completed. All previously identified issues have been successfully resolved:

  • Fixed invalid CSS class text-shrink-0 in TodoListDisplay.tsx (now shrink-0)
  • Fixed invalid Tailwind classes -mt-1.25 and max-w-50 in TaskHeader.tsx (now -mt-1.5 and max-w-md)
  • Removed unused CSS animation gradient-text from index.css
  • Removed unused prop isParentExpanded from TodoListDisplay component

No new issues found. The PR is ready to merge.

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


<div className="pl-1 pr-1 pt-1 font-light leading-normal">
{todoGroups.map((group, index) => (
<TodoList key={index} {...group} />
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider using a stable unique key (e.g. group.keyPrefix) instead of the array index when mapping over todoGroups. This helps ensure consistent rendering across re-renders.

Suggested change
<TodoList key={index} {...group} />
<TodoList key={group.keyPrefix} {...group} />

{
todos: startedTodos,
status: "in_progress",
keyPrefix: "started",
Copy link
Contributor

Choose a reason for hiding this comment

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

There is an inconsistent naming for the keyPrefix. In the initial state branch the in-progress group uses keyPrefix 'in-progress', but in the update branch it uses 'started'. If both refer to in-progress tasks, consider aligning the naming or adding a comment to clarify the distinction.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 7, 2025
@mrubens mrubens merged commit fff5cf8 into main Nov 7, 2025
13 checks passed
@mrubens mrubens deleted the bb/visuals-todos branch November 7, 2025 22:26
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Nov 7, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Nov 7, 2025
mini2s added a commit to zgsm-ai/costrict that referenced this pull request Nov 9, 2025