Skip to content

fix(frontend): Ensure error and status messages wrap correctly within containers#13522

Merged
hieptl merged 2 commits intoOpenHands:mainfrom
sjathin:open-hands-12757
Mar 23, 2026
Merged

fix(frontend): Ensure error and status messages wrap correctly within containers#13522
hieptl merged 2 commits intoOpenHands:mainfrom
sjathin:open-hands-12757

Conversation

@sjathin
Copy link
Copy Markdown
Contributor

@sjathin sjathin commented Mar 22, 2026

Summary of PR

  • Standardize top-right toast notification width to 400px and enable aggressive word wrapping (break-word, anywhere) to handle long URLs and skipped filenames.
  • Fix typo in ErrorMessageBanner by replacing non-standard 'wrap-break-words' class with the correct Tailwind 'break-words' utility.
  • Refactor ChatStatusIndicator to allow vertical expansion (min-h-[31px]) and multi-line wrapping, while ensuring it takes the full width of the chat container.
  • Adjust status indicator line-height to 16px for better readability in multi-line states.

Demo Screenshots/Videos

Screenshot 2026-03-21 at 23 32 33 Screenshot 2026-03-22 at 00 20 16

Change Type

  • Bug fix
  • New feature
  • 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 #12757

Release Notes

  • Include this change in the Release Notes.

Jathin Sreenivas added 2 commits March 22, 2026 01:08
… containers

- Standardize top-right toast notification width to 400px and enable
aggressive word wrapping (break-word, anywhere) to handle long URLs and
skipped filenames.
- Fix typo in ErrorMessageBanner by replacing non-standard
'wrap-break-words' class with the correct Tailwind 'break-words'
utility.
- Refactor ChatStatusIndicator to allow vertical expansion
(min-h-[31px]) and multi-line wrapping, while ensuring it takes the full
width of the chat container.
- Adjust status indicator line-height to 16px for better readability in
multi-line states.
@sjathin sjathin marked this pull request as ready for review March 22, 2026 01:15
@sjathin sjathin requested review from amanape and hieptl as code owners March 22, 2026 01:15
Copy link
Copy Markdown
Collaborator

@enyst enyst 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 for the PR!

Please let me introduce your contribution to our FE channel on Slack and feel free to join: #proj-gui channel:
slack discussion

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.

Hello @sjathin,

I have reviewed and approved your pull request.

Thank you very much for your contribution! 🙏

@hieptl hieptl merged commit 1581b95 into OpenHands:main Mar 23, 2026
27 checks passed
@mamoodi mamoodi added the release:cloud-1.19.0 Included in release 1.19.0 label Mar 30, 2026 — with OpenHands AI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:cloud-1.19.0 Included in release 1.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Avoid toast for AgentErrorEvent

4 participants