Chat todos: revisit icons used for tasks and in progress (fix #273083)

[bpasero]

No description provided.

[Copilot]

Pull Request Overview

This PR updates the todo list widget to show a spinning loading icon for in-progress tasks and improves the "Done" message to display the task count consistently with other states.

Key changes:

• Modified the getStatusIconClass method to return an array of CSS classes instead of a single class, enabling support for multiple classes like the spinning animation modifier
• Updated the "Done" message to include the total count format for consistency

[bhavyaus]

This removes the icons from the header -> was this intentional?

Also, the spinner continues to animate after the response is completed or stopped, when there are in-progress todos.

That feels off to me.

We should consider showing a canceleld icon or reset its state to not-started in the widget to avoid showing a spinner when nothing is happening

[bpasero]

@bhavyaus

This removes the icons from the header -> was this intentional?

this was only done so that todos - when collapsed - look similar to the edit working set below since that one also does not have an icon. We could play with having the icon to the right though.

Also, the spinner continues to animate after the response is completed or stopped, when there are in-progress todos.

That is a good point, but that was already the case before, with the previous in-progress icon, even though not animated. Do we know when the response stopped and could track that as another state in the todo widget?

[bpasero]

Also not very pretty:

This is not ready, closing for now.