Content-Length: 377967 | pFad | https://github.com/All-Hands-AI/OpenHands/pull/11202

1B fix(frontend): improve chat input status indicator responsiveness by YakshithK · Pull Request #11202 · All-Hands-AI/OpenHands · GitHub
Skip to content

Conversation

YakshithK
Copy link
Contributor

  • This change is worth documenting at https://docs.all-hands.dev/
  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality this introduces.

This fix improves the responsiveness of the chat input status indicator in OpenHands, ensuring it displays correctly even when the window size is reduced. This resolves the UI issue where the indicator layout broke or appeared incorrectly on smaller screens.

Screenshot 2025-09-25 at 4 26 18 PM

Summarize what the PR does, explaining any non-trivial design decisions.

This PR adjusts the styling and layout of the waiting-for-runtime-message.tsx component to handle various viewport sizes gracefully. The changes involve CSS adjustments and minor component structure modifications to improve adaptability without affecting other UI elements. The solution ensures consistent behavior across devices and window sizes while maintaining performance and accessibility.


Link of any specific issues this addresses:

Fixes #11117

@YakshithK YakshithK requested a review from amanape as a code owner October 1, 2025 04:31
@hieptl
Copy link
Collaborator

hieptl commented Oct 1, 2025

Hi @YakshithK, I found an issue when testing the PR locally.

issue: When accessing the conversation page on a large screen, the text does not remain on a single line but instead wraps into two lines. Subsequently, when the chat window is resized to a smaller width, the display issue persists with little noticeable difference from the initial state. For further clarification, please refer to the video provided below.

issue.mov

@YakshithK
Copy link
Contributor Author

Thanks for catching this! I updated the CSS to ensure the status text stays single-line on large screens and adapts properly when resizing. Let me know if you catch anything else!

Copy link
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.

Thank you! 🙏

@amanape amanape enabled auto-merge (squash) October 2, 2025 14:43
@amanape
Copy link
Member

amanape commented Oct 2, 2025

Thank you!

auto-merge was automatically disabled October 2, 2025 16:15

Head branch was pushed to by a user without write access

@YakshithK YakshithK requested a review from mamoodi as a code owner October 2, 2025 16:15
@YakshithK
Copy link
Contributor Author

Hi all, the Python test is failing on Linux, but I believe it’s unrelated to my frontend changes. Let me know if you want me to look into it, otherwise this PR is ready to merge.

@hieptl hieptl merged commit aab6f41 into All-Hands-AI:main Oct 2, 2025
35 of 39 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.

[Bug]: Chat input status indicator’s don’t look great if window is too small

7 participants









ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/All-Hands-AI/OpenHands/pull/11202

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy