search

emmanuelsw / notiwind

A headless Vue 3 notification library to use with Tailwind CSS.
https://notiwind-demo.netlify.app
MIT License
277 stars 28 forks source link

Immediate effect max notifications #41

Closed Tamas-hi closed 1 year ago

Tamas-hi commented 1 year ago

With regards of #40 , I played around a bit with the source code and found a possible solution for the problem. Basically I made sure that state.notifications can't hold more notification than the value of props.maxNotifications.

Let me know if this is good enough for merge, or if there is anything else left to be added here.

emmanuelsw commented 1 year ago

Hey! thank you for the PR, I tested it and it works great. Can you fix the conflicts before merging it?

Tamas-hi commented 1 year ago

Hi! Thanks! It should be good to go now.