feat(Notification): add `pauseTimeoutOnHover` prop

HarshPatel5940 commented 1 week ago

🔗 Linked issue

Not linked to an issue.

❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This pull request includes changes to the Notification component to add a new feature and improve user interaction. The most important changes include adding a new prop to control the pause behavior on hover and updating the mouse event handlers to respect this new prop.

Enhancements to Notification component:

src/runtime/components/overlays/Notification.vue: Added a new pauseTimeoutOnHover prop to allow control over whether the timeout is paused when the notification is hovered.
src/runtime/components/overlays/Notification.vue: Updated onMouseover and onMouseleave methods to check the new pauseTimeoutOnHover prop before pausing or resuming the timer.

📝 Checklist

[ ] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

HarshPatel5940 commented 1 week ago

More context, this PR will allow users to perform toasts which don't stop the timeout when we hover.

My field name might be improvised, please suggest me better if you have. I really needed this flexibility inside the toast so i have made a PR.

sandros94 commented 1 week ago

I was unsure which part in the documentation you would like to be updated?

You can add that information here, with the related boolean prop: https://github.com/nuxt/ui/blob/d0e626c55185ba93dd07eb71167e3e6aeed4a013/docs/content/2.components/notification.md?plain=1#L138-L153

HarshPatel5940 commented 6 hours ago

Thank you boss!

nuxt / ui