Closed jumtp closed 1 month ago
Someone is attempting to deploy a commit to the Themesberg Team on Vercel.
A member of the Team first needs to authorize it.
The update in Badge.svelte
enhances the component's event handling by integrating createEventDispatcher
. This allows the badge's close button to dispatch a custom 'close' event, aligning with the need for improved callback functionality in dismissible badges.
File | Change Summary |
---|---|
src/lib/badge/Badge.svelte |
Updated import to include createEventDispatcher , added dispatcher for 'close' event on click. |
Objective | Addressed | Explanation |
---|---|---|
Badge close callback is not working (#1320) | β |
π°π In the code where the badges dwell, A tiny fix makes all work well. Click! A badge now says goodbye, With a custom event flying high. CodeRabbit hops off, job well done, Another bug squashed, another victory won! π₯π
src/lib/badge/Badge.svelte (2)
`3-3`: Import of `createEventDispatcher` correctly added for handling close events. --- `16-16`: Creation of `dispatcher` using `createEventDispatcher()` is appropriate for custom event dispatching.
The latest updates on your projects. Learn more about Vercel for Git βοΈ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
flowbite-svelte | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Apr 21, 2024 11:51am |
Thank you for your contribution.
Closes #1320 Closes #1321
π Description
TransitionFrame Close callback is correct for Badge.I think Badge Close callback should use createEventDispatcher.because multiple component use TransitionFrame.The TransitionFrame Close callback is not sure work for everycomponent.
Status
β Checks
βΉ Additional Information
https://github.com/themesberg/flowbite-svelte/assets/49558643/ad49ec00-c855-451a-968c-152c00da0dd3
Summary by CodeRabbit