Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

[Feature] Prevent InAppNotification from disappearing when hovered #4194

Closed
vgromfeld opened this issue Aug 24, 2021 · 3 comments
Closed

[Feature] Prevent InAppNotification from disappearing when hovered #4194

vgromfeld opened this issue Aug 24, 2021 · 3 comments
Assignees
Labels
accessibility ♿ feature request 📬 A request for new changes to improve functionality improvements ✨
Milestone

Comments

@vgromfeld
Copy link
Contributor

Describe the problem this feature would solve

The InAppNotification control is automatically hidden after the expiration of the notification duration. The durations we are hardcoding in our applications may not be long enough to allow the users to fully read the message. We should have a way to keep the notification visible for a longer time if the user wants/needs it.

Describe the solution

Windows toast notifications remains displayed when they are hovered by the mouse cursor (or tapped). We should replicate this feature in our InAppNotification control.

We can add a new property to the control to enable (default value) or disable this feature.

@vgromfeld vgromfeld added the feature request 📬 A request for new changes to improve functionality label Aug 24, 2021
@ghost
Copy link

ghost commented Aug 24, 2021

Hello, 'vgromfeld! Thanks for submitting a new feature request. I've automatically added a vote 👍 reaction to help get things started. Other community members can vote to help us prioritize this feature in the future!

@michael-hawker michael-hawker added this to the 7.2/8.0? milestone Aug 25, 2021
@michael-hawker
Copy link
Member

Great idea @vgromfeld is this something you're thinking of looking at in the future? We're just about to close 7.1, so assuming this would be for the future?

I know it'd be good for us to encapsulate the base InfoBar for the rendered portion of our InAppNotification (and basically have our control more about the templating and logic for stacked notifications). Not sure how these two things interact/conflict from an implementation stand-point.

@vgromfeld
Copy link
Contributor Author

Yes, I will try to implement it. I can have a quick look to InfoBar and see how/if we can merge the two controls.

We're just about to close 7.1, so assuming this would be for the future?
Yes, I will probably not have the time to complete the implementation before 7.1 release.

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
accessibility ♿ feature request 📬 A request for new changes to improve functionality improvements ✨
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants