Skip to content

Conversation

ryu-man
Copy link
Contributor

@ryu-man ryu-man commented Sep 2, 2025

Addresses #4084

  • switch from textarea to milkdown to handle user input
  • set max height & overflow scroll when having long content
  • improve transition from center to bottom state

@ryu-man ryu-man requested a review from ivyjeong13 September 3, 2025 16:46
@ryu-man ryu-man marked this pull request as ready for review September 3, 2025 16:46
>
<!-- <div class="max-h-full min-h-full w-full flex-1 overflow-y-auto"></div> -->

<div class="flex min-h-full w-full flex-col" {id}>
Copy link
Contributor

@ivyjeong13 ivyjeong13 Sep 4, 2025

Choose a reason for hiding this comment

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

Can you check if the / command to show prompts work with this input change? Wasn't popping up for me on this branch but did see it pop up in main:

Screenshot 2025-09-04 at 11 35 44 AM

Would also check that it works and supplies the prompt into the input after selecting one of them and filling out any fields if it requires it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wow, I did not know this feature existed. Going to fix any issues

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Contributor

Choose a reason for hiding this comment

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

https://www.loom.com/share/3ccc536345154077b43e0293f892675f

I see prompts popping up! But it doesn't look to supply the filled out prompt after filling out the information? Switched to main to give an example in the Loom if it helps!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the loom; I will address that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryu-man ryu-man requested a review from ivyjeong13 September 10, 2025 16:15
@ryu-man ryu-man merged commit ddd998d into obot-platform:main Sep 19, 2025
4 checks passed
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