Skip to content

Comments

docs: add message list attachments documentation#5207

Open
tomivirkki wants to merge 3 commits intomainfrom
docs/message-list-item-attachments
Open

docs: add message list attachments documentation#5207
tomivirkki wants to merge 3 commits intomainfrom
docs/message-list-item-attachments

Conversation

@tomivirkki
Copy link
Member

Add message list attachments documentation

@tomivirkki tomivirkki marked this pull request as draft February 23, 2026 14:05
<>
<MessageList
items={items}
onAttachmentClick={(e) => {
Copy link
Member Author

Choose a reason for hiding this comment

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

note: this doesn't work currently, but will be fixed by vaadin/web-components#11189

Copy link
Contributor

Choose a reason for hiding this comment

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

The Lit example does not seem to work for me at all.

Copy link
Member Author

Choose a reason for hiding this comment

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

The Lit example does not seem to work for me at all.

Works fine for me. Try cleaning up and restarting.

Kapture.2026-02-25.at.12.00.44.mp4

@tomivirkki tomivirkki force-pushed the docs/message-list-item-attachments branch 3 times, most recently from 9a43711 to 243cdcf Compare February 24, 2026 12:23
@tomivirkki tomivirkki force-pushed the docs/message-list-item-attachments branch from 243cdcf to 3a639f1 Compare February 24, 2026 12:28
@tomivirkki tomivirkki marked this pull request as ready for review February 24, 2026 12:42
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