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

[Dashboard] Notifications Page #3398

Closed
gtsiolis opened this issue Mar 9, 2021 · 1 comment
Closed

[Dashboard] Notifications Page #3398

gtsiolis opened this issue Mar 9, 2021 · 1 comment

Comments

@gtsiolis
Copy link
Contributor

gtsiolis commented Mar 9, 2021

Problem to solve

Currently, notifications settings are mixed with other user settings like Environment Variables, API Tokens, etc.

This is part of #3300. 🍕

User Experience goal

The user should be able to a navigate and update their notifications settings while the system confirms their action by showing a toast notification.

Proposal

In this first MVC (minimum viable change), we can do the following:

  1. Introduce a dedicated notifications settings page.
  2. Checking or unchecking the two existing options should trigger a toast notification at the bottom of the screen.

Components involved:

  1. Checkbox
  2. Toast

Designs

Notifications Notifications (Spacing)
image image

In the designs above, there're are also hover states for the top tabs and the sidebar tabs.

See also design specs.

Documentation

No documentation changes are needed for this first iteration.

@gtsiolis
Copy link
Contributor Author

Closing as this has been resolved in #3513. Added a follow-up issue in #3530.

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

No branches or pull requests

2 participants