-
Notifications
You must be signed in to change notification settings - Fork 60
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
Notification UI/UX update #5970
Notification UI/UX update #5970
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Waseem826 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
bb2c743
to
4fb3535
Compare
8a5659f
to
de9ed33
Compare
de9ed33
to
f767f9b
Compare
f767f9b
to
d091404
Compare
/lgtm |
LGTM label has been added. Git tree hash: fb3ac23c686cd53b538068e0d38f57336ea2433c
|
What this PR does / why we need it:
This PR brings several changes to Notification UI/UX:
New Design
Success Notification:
Short Error Notification:
Error notifications now display a short headline above the actual error. In case its some HTTP error then HTTP error status will be displayed. Otherwise, it will default to
Something went wrong
.Long Error Notification:
If error message is too long then it will be collapsed by default and user can expand it by clicking on down arrow icon.
Collapsed:
Expanded:
Notification Count
The bell icon in top nav bar now displays unread notifications count.
Other Changes
Which issue(s) this PR fixes:
Fixes #4662
What type of PR is this?
/kind design
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:
Documentation: