Skip to content

refactor: message attachment base styles and themes#11308

Merged
web-padawan merged 2 commits intomainfrom
refactor/message-attachment-styles
Mar 10, 2026
Merged

refactor: message attachment base styles and themes#11308
web-padawan merged 2 commits intomainfrom
refactor/message-attachment-styles

Conversation

@jouni
Copy link
Copy Markdown
Member

@jouni jouni commented Mar 10, 2026

Fix and clean up the base styles and themes for message attachments. Align the icon sizing with the sizing of the icon in upload-file thumbnail variant.

In Aura, it felt excessive to let the attachment buttons get all the same styles as vaadin-button.

@jouni jouni requested a review from tomivirkki March 10, 2026 12:26
Copy link
Copy Markdown
Member

@tomivirkki tomivirkki left a comment

Choose a reason for hiding this comment

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

LGTM, Didn't find any issues, Sonar found some commented out code. Are they left there intentionally?

@jouni jouni force-pushed the refactor/message-attachment-styles branch from 4af7781 to b452e27 Compare March 10, 2026 14:36
@jouni
Copy link
Copy Markdown
Member Author

jouni commented Mar 10, 2026

Removed the commented code (forgot to do that before pushing).

Copy link
Copy Markdown
Member

@web-padawan web-padawan left a comment

Choose a reason for hiding this comment

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

Let me add new custom CSS properties to the JSDoc for vaadin-message.

@sonarqubecloud
Copy link
Copy Markdown

@web-padawan web-padawan merged commit 72c4bcc into main Mar 10, 2026
10 checks passed
@web-padawan web-padawan deleted the refactor/message-attachment-styles branch March 10, 2026 15:09
@vaadin-bot
Copy link
Copy Markdown
Collaborator

This ticket/PR has been released with Vaadin 25.1.0-beta3.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants