Project

General

Profile

Bug #18193

When there are no notification events, the notification drawer should mention it

Added by Ohad Levy over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Notifications
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

steps to reproduce:

1. ensure there are no pending notifications to the user
2. click on the notification icon

see attached screenshot.

empty-notifications.png View empty-notifications.png 9.69 KB Ohad Levy, 01/22/2017 04:33 AM
Empty notifications

Related issues

Related to Foreman - Feature #18010: UI Notifications front-end implementation Closed2017-01-10

Associated revisions

Revision a29d5438 (diff)
Added by matan werbner over 4 years ago

Fixes #18957,#18193,#18678,#18853 - notifications redux

Refactored notifications to work with redux, fixed
several UI related bugs, added tests.
flux is now unused, this is based on @gailsteiger's
previous work.

Revision e2b7102e (diff)
Added by matan werbner over 4 years ago

Fixes #18957,#18193,#18678,#18853 - notifications redux

Refactored notifications to work with redux, fixed
several UI related bugs, added tests.
flux is now unused, this is based on @gailsteiger's
previous work.

History

#1 Updated by Ohad Levy over 4 years ago

  • Related to Feature #18010: UI Notifications front-end implementation added

#2 Updated by Ohad Levy over 4 years ago

  • Subject changed from Notification Icon should not do anything when clicking on it when there are no events to When there are no notification events, the notification drawer should mention it
  • Category set to Notifications

according to http://www.patternfly.org/pattern-library/communication/notification-drawer/#/_design item number 7:
Icon Empty: The empty state shows no new events.

#3 Updated by Gail Steiger over 4 years ago

  • Assignee set to Gail Steiger

#4 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4401 added

#5 Updated by matan werbner over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#6 Updated by Ohad Levy over 4 years ago

  • Assignee changed from Gail Steiger to matan werbner
  • Legacy Backlogs Release (now unused) set to 209

Also available in: Atom PDF