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.

Sign up for GitHub

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

Jump to bottom

[Notification]: Update Actionable notification (Toast style) to have the time-stamp prop #847

Open
3 tasks
alina-jacob opened this issue Jul 8, 2024 · 2 comments
Open
3 tasks

[Notification]: Update Actionable notification (Toast style) to have the time-stamp prop #847

alina-jacob opened this issue Jul 8, 2024 · 2 comments
Labels
component: notification kit: figma role: design ✏️ type: bug 🐛

Comments

@alina-jacob
Copy link

Actionable notification (Toast style)

Actionable notification with the Toast style needs to have an additional string prop / slot to show Time-stamp.

Quick fix:

Elaborate fix:

  • Revisit the Figma variants for notification, the ideal branching as per current docs should be
  • Inline (with and without interaction)
  • Toast (with and without interaction time-stamp)
@Kritvi-bhatia17
Copy link

Thanks for creating the issue @alina-jacob!

A possible Figma solution could involve initially displaying all layers, including Time stamp and Actionable components. Additionally, users could have the option to toggle these features on or off based on their specific use case.
This approach would help reduce variants by combining two variants (time stamp and actionable) into one.
(Attaching a video for reference would help illustrate this further.)

Screen.Recording.2024-07-09.at.3.15.10.AM.mov

@Kritvi-bhatia17
Copy link

Related: thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: notification kit: figma role: design ✏️ type: bug 🐛
Projects
Design System
Status: 🪆 Needs Refined
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Kritvi-bhatia17 @alina-jacob