Skip to content

style(ui): align chat messages from the top #719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

jayolee
Copy link
Contributor

@jayolee jayolee commented May 30, 2025

Updates

  • Updated flex direction of messages to align from top so the user can read the generations while streaming

Notes to reviewer

Scroll to bottom button is not currently displayed due to another issue(#718)

Contributes to: #664

@jayolee jayolee self-assigned this May 30, 2025
- Updated flex direction of messages to align them from the top

Contributes to: i-am-bee#664
Signed-off-by: jayolee <[email protected]>
@jayolee jayolee force-pushed the style-message-alignment-change branch from cc539d4 to 48f08ec Compare May 30, 2025 19:28
@kapetr kapetr self-requested a review June 2, 2025 15:09
@kapetr
Copy link
Contributor

kapetr commented Jun 2, 2025

@jayolee I’m sorry, but I can’t approve this as the solution doesn’t align with our design - messages are aligned to the bottom. Users should, of course, be able to read the streamed response as soon as it appears. If that’s not happening, it’s a bug that needs to be addressed.

@kapetr
Copy link
Contributor

kapetr commented Jun 3, 2025

@jayolee FYI #664 (comment)

@jayolee
Copy link
Contributor Author

jayolee commented Jun 3, 2025

Hi @kapetr,. I'll close this for now as the new design is coming next week. Thank you!

@jayolee jayolee closed this Jun 3, 2025
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.

2 participants