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

Chat UI quality of life improvements #154

Merged
merged 2 commits into from
May 17, 2023

Conversation

JasonWeill
Copy link
Collaborator

Fixes #55 (remaining tasks)

Adds a placeholder message for the chat box:

placholder-message

Adds a tooltip for the send message button:

send-message-tooltip

Updates a screen shot in the docs.

@JasonWeill JasonWeill added enhancement New feature or request @jupyter-ai/chatui labels May 8, 2023
Copy link
Member

@dlqqq dlqqq left a comment

Choose a reason for hiding this comment

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

It looks like this PR also includes many other changes from other branches; could you rebase the pertinent commits back onto main? For example:

git checkout main
git pull
git checkout -
git rebase HEAD~3 chat-ui-tweaks --onto main
git push -f

@JasonWeill
Copy link
Collaborator Author

@dlqqq Revised to exclude changes in #136.

Copy link
Collaborator

@andrii-i andrii-i left a comment

Choose a reason for hiding this comment

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

This looks and works well.

At the same time if we want Jupyternaut to be seen as another collaborator in RTC chat environment, more neutral chat input placeholder at L48 like "Send a message" would work better. Call to action can then be part of a placeholder (as well as instructions and example prompts) that is shown when there are no messages in the chat (separate PR). I might be missing context here so feel free to ignore.

Copy link
Member

@dlqqq dlqqq left a comment

Choose a reason for hiding this comment

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

This looks great! Awesome work @JasonWeill 🎉

@andrii-i These are good suggestions, but right now, the AI always responds to the user. So the placeholder "Ask Jupyternaut anything" makes sense. Once we implement multi-user rooms that require the AI to be mentioned before responding, then we can change the placeholder. 👍

@dlqqq dlqqq merged commit b973da0 into jupyterlab:main May 17, 2023
dbelgrod pushed a commit to dbelgrod/jupyter-ai that referenced this pull request Jun 10, 2024
* Updates screen shot with placeholder

* Incorporates change to code
Marchlak pushed a commit to Marchlak/jupyter-ai that referenced this pull request Oct 28, 2024
* Updates screen shot with placeholder

* Incorporates change to code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request @jupyter-ai/chatui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

better Chat UI input
3 participants