Skip to content

Conversation

@IvanDanchev
Copy link
Contributor

@IvanDanchev IvanDanchev commented Nov 5, 2025

@IvanDanchev IvanDanchev requested review from a team as code owners November 5, 2025 09:49
@IvanDanchev IvanDanchev added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Nov 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

Hello @IvanDanchev,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@IvanDanchev IvanDanchev removed the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Nov 5, 2025
Comment on lines +28 to +30
* `ChatMessageFilesLayoutMode.Vertical`—Files are displayed in a vertical stack (default)
* `ChatMessageFilesLayoutMode.Horizontal`—Files are displayed in a horizontal row
* `ChatMessageFilesLayoutMode.Wrap`—Files wrap to the next line when they exceed the message width
Copy link
Contributor

Choose a reason for hiding this comment

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

To avoid maintenance in the future, I suggest removing the list of available options and adding a link to the API reference: https://www.telerik.com/blazor-ui/documentation/api/telerik.blazor.components.chatmessagefileslayoutmodemode

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In general I agree, but in this case we've likely added all viable layout modes and I don't expect we are going to be adding more.

@IvanDanchev IvanDanchev merged commit 6f6f3cd into master Nov 10, 2025
@github-actions github-actions bot deleted the chat-new-features branch November 10, 2025 14:39
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.

4 participants