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

Don't hide bell-icon when there are no notifications #107

Closed
wants to merge 1 commit into from

Conversation

MariusBluem
Copy link
Member

  • Discussion Consider about showing Notifications  #2
  • Makes sense now, since we have a lot of notifications now (Talk and other Collaborative Features), and we want to promote the notifications-feature more. Other services do this the same way ;)

@jancborchardt @nickvergessen @nextcloud/designers

Signed-off-by: Marius Blüm marius@lineone.io

Signed-off-by: Marius Blüm <marius@lineone.io>
@nickvergessen
Copy link
Member

#95 has this included, together with a proper empty content message etc.
I will rebase it this week, and we should then go with that instead.

@nickvergessen nickvergessen deleted the show-bell-icon-always branch January 3, 2018 11:06
@MariusBluem
Copy link
Member Author

#95 has this included, together with a proper empty content message etc.
I will rebase it this week, and we should then go with that instead.

👍

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

Successfully merging this pull request may close these issues.

2 participants