Skip to content

Conversation

@wsxiaoys
Copy link
Member

@wsxiaoys wsxiaoys commented Oct 16, 2025

Summary

  • Move ErrorMessageView to ChatToolbar for better positioning
  • Add proper spacing for error messages with absolute positioning
  • Update component imports and layout adjustments

Screen recording

https://jam.dev/c/e0e41da7-7d98-4331-b162-0d1a33b9d0df

Screenshot

image

🤖 Generated with Pochi

@wsxiaoys wsxiaoys requested a review from liangfung October 16, 2025 18:46
@wsxiaoys wsxiaoys marked this pull request as draft October 21, 2025 04:52
@wsxiaoys
Copy link
Member Author

Pending issue - error message might go over single line, height cannot be pre-reserved.

@wsxiaoys wsxiaoys moved this to Todo in Pochi Tasks Oct 27, 2025
@liangfung liangfung moved this from Todo to In Progress in Pochi Tasks Nov 17, 2025
wsxiaoys and others added 2 commits November 17, 2025 15:02
- Move ErrorMessageView to ChatToolbar for better positioning
- Add proper spacing for error messages with absolute positioning
- Update component imports and layout adjustments

🤖 Generated with [Pochi](https://getpochi.com)

Co-Authored-By: Pochi <[email protected]>
@liangfung liangfung force-pushed the improve-chat-error-handling branch from 2fbf4d8 to 9272224 Compare November 17, 2025 18:10
@liangfung liangfung marked this pull request as ready for review November 18, 2025 03:40
@liangfung liangfung requested a review from zhanba November 18, 2025 03:40
@liangfung liangfung force-pushed the improve-chat-error-handling branch from 23df98e to 3e6aed2 Compare November 18, 2025 04:01
@zhanba zhanba added this pull request to the merge queue Nov 18, 2025
Merged via the queue into main with commit e02d3d5 Nov 18, 2025
3 checks passed
@zhanba zhanba deleted the improve-chat-error-handling branch November 18, 2025 05:41
@github-project-automation github-project-automation bot moved this from In Progress to Done in Pochi Tasks Nov 18, 2025
@wsxiaoys wsxiaoys moved this from Done to Verified in Pochi Tasks Nov 20, 2025
@wsxiaoys wsxiaoys moved this from Verified to Archived in Pochi Tasks Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Archived

Development

Successfully merging this pull request may close these issues.

4 participants