-
-
Notifications
You must be signed in to change notification settings - Fork 55
feat(badge): new component #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for anu-vue ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Hi 👋🏻 Thanks for PR. This is huge ❤️ I really loved the idea of the I have a few changes if you don't mind correcting them. Please let me know if you need any help, I will be there for you. |
Hey thanks for your detailed feedback completely agree, will get this done soon as I'm on vacation at the moment. Talk to you soon |
Hey i did some updates based on your feedback hopefully it's better However about this one i am not sure what it is yet 🤔
|
Hi, Thanks for taking the time to update the code. Sorry I was busy with the list component lately. I will start reviewing it tomorrow. I hope you enjoyed your vacation 🏖️ |
There's good news for you 😇 If we merge #16 you can omit writing |
I reviewed the PR and everything is exactly how it should be ❤️ I have introduced the new Moreover, I have fetched the latest code update from the main branch. The last thing I need help with is that we introduced Can you help me fix that? Thanks 😇 |
I will also update the |
Will take a look at that prop soon |
We are planning to make |
Hi, today i worked on a Badge component to wrap other elements, hope you like it. Did my best to try to follow the current standards but feel free to change anything