Skip to content
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

Scroll to bottom on new message, allow new lines #58392

Merged
merged 2 commits into from
May 6, 2024

Conversation

bencodeorg
Copy link
Contributor

@bencodeorg bencodeorg commented May 2, 2024

Two unrelated improvements to the chat UI:

  • scroll to the bottom of the chat window when a new message is appended (currently, it doesn't scroll, and you have to manually scroll to the bottom)
  • don't remove newlines that are included in the model response

Links

Before (newlines stripped) After (newlines included)
image image
autoscroll.mov

Testing story

Tested manually (see video/screenshots above). I also noticed that this change preserves newlines in a user message as well (ie, if you hit shift+enter while entering a message, that newline is reflected in the chat workspace)

@bencodeorg bencodeorg requested a review from a team May 3, 2024 23:56
Copy link
Contributor

@fisher-alice fisher-alice left a comment

Choose a reason for hiding this comment

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

Cool! Just a left a question for my own understanding. Thanks!

apps/src/aichat/views/chatMessage.module.scss Show resolved Hide resolved
@bencodeorg bencodeorg merged commit 9559342 into staging May 6, 2024
2 checks passed
@bencodeorg bencodeorg deleted the ben/gen-ai-scroll-to-bottom branch May 6, 2024 17:28
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.

None yet

2 participants