Skip to content
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

Reduce the Notification-Message display timings #8157

Closed
gayann-au opened this issue Jul 16, 2024 · 4 comments · Fixed by #8689
Closed

Reduce the Notification-Message display timings #8157

gayann-au opened this issue Jul 16, 2024 · 4 comments · Fixed by #8689

Comments

@gayann-au
Copy link

Error Messages/Notifications when attempted to update data, access denial Notification, Notifications when Discussion Notes are filing attempts are done and for other Error Messages, the Notification message display timings should be reduced in contrast to current usage. Currently these message take extra Time to Auto Vanish.

Attached few snapshots of these messages for ref:
image
image
image

Copy link

Hi, @coronasafe/care-frontend-maintainers, This issue has been automatically marked as stale because it has not had any recent activity.

@github-actions github-actions bot added the stale label Jul 31, 2024
@gigincg gigincg added good first issue Good for newcomers and removed stale labels Aug 12, 2024
@gigincg
Copy link
Member

gigincg commented Aug 12, 2024

Anyone who would like to work on it, please describe your solution before working on it

@github-actions github-actions bot added the stale label Aug 29, 2024
@HitishRaoP
Copy link
Contributor

HitishRaoP commented Sep 10, 2024

@gigincg By adding a delay option in the notify configuration, we can control the visibility duration of notifications. This can be set to a reasonable default (e.g., 1000 milliseconds), while still allowing users to adjust it based on their requirements.

File : https://github.com/HitishRaoP/care_fe/blob/develop/src/Utils/Notifications.js

Relevant code:
Screenshot 2024-09-10 114610

@BarryByte
Copy link

@gigincg Hello maintainers, i would like work on this issue, please assign this issue to me
My solution is to add include a delay option in the notification settings allows control over how long notifications are displayed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants