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

[Callout] Explore ways of adding this feature into our current system structure #17024

Closed
2 tasks done
Tracked by #15747
laurenmrice opened this issue Jul 23, 2024 · 1 comment
Closed
2 tasks done
Tracked by #15747
Assignees
Milestone

Comments

@laurenmrice
Copy link
Member

laurenmrice commented Jul 23, 2024

Acceptance criteria

  • In this issue, we need to look at our current Notification component (Actionable, Inline, Toast) and explore ways of how we could currently incorporate the Static Notification feature parameters without making a net new component.
  • Note: Alternatively to the approach in this issue, we are exploring an additional option of creating a net new component for this work, which is being explored in this separate issue: (Callout): Design iterations #16984

Tasks

  • Explore how we could leverage our current structure of the Notification component and incorporate the needs of StaticNotification.

    • For example:
      • Could we add a new variant to Notification that would solve these needs?
      • Could we adjust one of our current Notification variants to solve these needs?
      • Could it just be an added prop on a current variant?
      • Other ideas?
  • Playback to Design / Dev / A11y

@laurenmrice laurenmrice added this to the 2024 Q3 milestone Jul 23, 2024
@laurenmrice laurenmrice changed the title [Static Notification] Explore how to address this feature in our current system structure [Static Notification] Explore ways of adding this feature into our current system structure Jul 23, 2024
@benjamin-kurien
Copy link

Multiple ways of documenting the component within our current system was explored and discussed with the team. For the current scope it was decided to explore the component as a 4th variant within or notification component.

@laurenmrice laurenmrice changed the title [Static Notification] Explore ways of adding this feature into our current system structure [Callout] Explore ways of adding this feature into our current system structure Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants