Skip to content

Conversation

khushal87
Copy link
Member

@khushal87 khushal87 commented Aug 20, 2025

The goal of the PR is to optionally override the Textinput component inside the AutoCompleteInput component using a component prop to the MessageInput component.

Note: Also changed the deprecated types to the useful ones.

@khushal87 khushal87 requested a review from isekovanic August 20, 2025 12:49
@Stream-SDK-Bot
Copy link
Contributor

Stream-SDK-Bot commented Aug 20, 2025

SDK Size

title develop branch diff status
js_bundle_size 284 KB 284 KB 0 B 🟢

);

if (TextInputComponent) {
return <TextInputComponent {...rest} />;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please make sure the passed props are the same here, so that the overidden component can be used directly and integrators can decide what to keep and what to remove. Otherwise, stuff like autoFocus, multiline and what not will break (and so will our UI).

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@isekovanic isekovanic left a comment

Choose a reason for hiding this comment

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

Please check the comment I left there, otherwise looks good !

Copy link
Contributor

@isekovanic isekovanic left a comment

Choose a reason for hiding this comment

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

Looks good ! Thank you !

@khushal87 khushal87 merged commit bb61233 into develop Aug 20, 2025
5 checks passed
@khushal87 khushal87 deleted the feat/override-text-input-completely branch August 20, 2025 13:45
@github-actions github-actions bot mentioned this pull request Aug 20, 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.

3 participants