Skip to content

Replace conversation loading spinner with skeleton cards#12230

Merged
amanape merged 6 commits intoOpenHands:mainfrom
adithya-naik:feat/conversation-card-skeleton
Jan 7, 2026
Merged

Replace conversation loading spinner with skeleton cards#12230
amanape merged 6 commits intoOpenHands:mainfrom
adithya-naik:feat/conversation-card-skeleton

Conversation

@adithya-naik
Copy link
Copy Markdown
Contributor

Summary of PR

This PR replaces the loading spinner in the conversation panel with skeleton cards to improve the user experience while conversations are loading. Instead of a single spinner, users now see placeholder skeletons for each conversation card, providing a more visually structured and responsive interface. Minor adjustments were also made in conversation-panel.tsx to support the skeleton card display logic.

Change Type

  • New feature
  • Bug fix
  • Breaking change
  • Refactor
  • Other (dependency update, docs, typo fixes, etc.)

Checklist

  • I have read and reviewed the code and I understand what the code is doing.
  • I have tested the code to the best of my ability and ensured it works as expected.

Fixes

Resolves #12194

Release Notes

  • Include this change in the Release Notes.

End-user friendly description:
Conversation loading now shows skeleton cards instead of a spinner, giving a faster perceived loading experience and improving visual feedback for users.

Copy link
Copy Markdown
Collaborator

@hieptl hieptl left a comment

Choose a reason for hiding this comment

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

Image

Hello @adithya-naik, Could you clarify why this checkbox was not selected? It appears that the code may not have been fully tested to ensure it works as expected.

@adithya-naik
Copy link
Copy Markdown
Contributor Author

Image Hello @adithya-naik, Could you clarify why this checkbox was not selected? It appears that the code may not have been fully tested to ensure it works as expected.

@hieptl
I wasn’t fully sure about the required test cases, which is why I did not check the testing checkbox.

@hieptl
Copy link
Copy Markdown
Collaborator

hieptl commented Jan 5, 2026

Hello @adithya-naik,

I found a few issues while testing your branch locally.

Issue 1: The Repository section and the Time should appear on the same line. Please refer to the image below for more details.

issue1

Thank you! 🙏

@adithya-naik
Copy link
Copy Markdown
Contributor Author

Solves #12194

@amanape made requested changes

@adithya-naik adithya-naik requested a review from amanape January 7, 2026 15:21
Copy link
Copy Markdown
Contributor

@amanape amanape left a comment

Choose a reason for hiding this comment

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

I made some additional style fixes.

This is a big UI improvement, thank you!

@amanape amanape enabled auto-merge (squash) January 7, 2026 15:29
@amanape amanape merged commit 15836c4 into OpenHands:main Jan 7, 2026
24 checks passed
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.

Replace loading spinner with skeleton cards in conversation panel

3 participants