⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@ddudenin
Copy link
Contributor

@ddudenin ddudenin commented Dec 24, 2025

Plan

  • Implement new component.
  • Add Avatar shape property.
  • Pass tests.
    • Compilation.
    • Check, clippy tests.
    • Doc comments.
    • Playwright.

Fixes #72

@ddudenin ddudenin marked this pull request as ready for review January 5, 2026 08:48
@ddudenin
Copy link
Contributor Author

ddudenin commented Jan 5, 2026

@ealmloff Happy New Year
Santa Claus put a new component under the tree as a gift for Dioxus ;)
Please take a look when you have time.

The test, as I understand it, fails because the new component isn't in the main branch.

@ealmloff ealmloff self-requested a review January 8, 2026 14:05
Copy link
Member

@ealmloff ealmloff left a comment

Choose a reason for hiding this comment

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

Thanks for working on this. I would like to support something closer to shadcn's badge component in the component library that generalizes outside of notifications which would require some changes

@ddudenin
Copy link
Contributor Author

@ealmloff I've made changes according to all the requests.

To maintain the functionality of the first version, I've separated the "Notifications" component into a separate type (based on the Badge).

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.

Badge component

2 participants