Skip to content

feat: add chat-input#948

Merged
timowolf merged 3 commits intomainfrom
feat/add-chat-input-in-chat-messages
Nov 5, 2025
Merged

feat: add chat-input#948
timowolf merged 3 commits intomainfrom
feat/add-chat-input-in-chat-messages

Conversation

@Killusions
Copy link
Copy Markdown
Member

@Killusions Killusions commented Nov 1, 2025

Add the chat-input component, needed for #600, uses #877 (merged)

Automatically expands based on text input (as per UX specs).

The actions and the disclaimer can also be provided in a declarative way.


@Killusions Killusions requested a review from a team as a code owner November 1, 2025 00:41
Copilot AI review requested due to automatic review settings November 1, 2025 00:41
@Killusions Killusions requested a review from kfenner November 1, 2025 00:41

This comment was marked as outdated.

@Killusions Killusions force-pushed the feat/add-chat-input-in-chat-messages branch from 817f7c7 to 02f0ad8 Compare November 1, 2025 00:56
@Killusions
Copy link
Copy Markdown
Member Author

Killusions commented Nov 1, 2025

@kfenner Should be ready now (but merge after the base chat messages because of the attachment list), the actions and the disclaimer can also be provided in a declarative way, automatically expands based on text input (as per UX specs), the default legal AI disclaimer will be defaulted in the (ai) chat container, as this input is not AI specific

@Killusions Killusions force-pushed the feat/add-chat-input-in-chat-messages branch 2 times, most recently from 0752028 to 8582142 Compare November 1, 2025 01:03
@Killusions Killusions mentioned this pull request Nov 1, 2025
1 task
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 1, 2025

@Killusions Killusions force-pushed the feat/add-chat-input-in-chat-messages branch 5 times, most recently from 9928527 to f098345 Compare November 4, 2025 18:09
@Killusions
Copy link
Copy Markdown
Member Author

@spike-rabbit @timowolf Anyone of you time to give it a review here? The actions and the disclaimer can be provided both in a model driven (should be used in most cases to follow UX guidelines) and in a declarative way (extensibility for custom actions like model selectors and custom disclaimer displaying)

@timowolf timowolf self-assigned this Nov 5, 2025
Copy link
Copy Markdown
Member

@timowolf timowolf left a comment

Choose a reason for hiding this comment

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

Fantastic.

@timowolf timowolf force-pushed the feat/add-chat-input-in-chat-messages branch from f098345 to 147c7ca Compare November 5, 2025 08:36
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 5, 2025

Code Coverage

@timowolf timowolf enabled auto-merge (rebase) November 5, 2025 09:21
@timowolf timowolf merged commit c06df6f into main Nov 5, 2025
11 checks passed
@timowolf timowolf deleted the feat/add-chat-input-in-chat-messages branch November 5, 2025 09:24
@siemens-element-bot
Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants