Skip to content

Conversation

Gyurmatag
Copy link
Collaborator

@Gyurmatag Gyurmatag commented Apr 29, 2025

This pull request intended to solve THIS issue.
So the input now support multiple lines just like ChatGPT. I've got some idea from Vercel's Chat SDK about the look and functionality.

Here is a screenshot for reference:
Képernyőfotó 2025-04-29 - 13 56 58

The file attachment button is disabled until we don't have the file attachment feature.

@threepointone
Copy link
Collaborator

could you fix Textarea to have support for dark mode as well, like all the other componens? toggle dark mode with the switch on top to check

@Gyurmatag
Copy link
Collaborator Author

could you fix Textarea to have support for dark mode as well, like all the other componens? toggle dark mode with the switch on top to check

@threepointone thank you for the heads up. It is fixed now!

@threepointone
Copy link
Collaborator

looks good. can you remove the attachment button altogether until we land the feature?

@Gyurmatag
Copy link
Collaborator Author

looks good. can you remove the attachment button altogether until we land the feature?

Of course, @threepointone . Done!

@threepointone
Copy link
Collaborator

the border around the textarea is too dark (in light mode) can you adjust it to match the rest of the ui?

tagging @stritt here since this adds a new component, will merge after his goahead

@Gyurmatag
Copy link
Collaborator Author

@threepointone good point. Done it. Looking for @stritt approval too!

@Gyurmatag Gyurmatag self-assigned this Apr 30, 2025
@Gyurmatag Gyurmatag requested a review from Brayden April 30, 2025 12:55
@Gyurmatag
Copy link
Collaborator Author

@Brayden I've done the changes. Let me know if you spot anything else!

@Gyurmatag Gyurmatag requested a review from Brayden May 1, 2025 10:16
@threepointone threepointone merged commit af97d5b into cloudflare:main May 4, 2025
1 check 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.

3 participants