-
Notifications
You must be signed in to change notification settings - Fork 14
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
Notifications icon shows by default in top bar #65
Comments
This is because we now have something that could show up: federated shares. |
When you do not receive a federated share, then it’s totally fine if it never shows up. If we want to advertise stuff to developers then we should do that, and not push it out to every single user. Especially not where space is so valuable such as in the top bar. Sorry but at the moment the cases where actually notifications will show up there are so few that it doesn’t warrant showing a yet unneeded icon for an empty dropdown the whole time. cc @karlitschek @owncloud/designers |
No strong opinions here from me. I can see the point that we shouldn't show an icon if it is of no use. On the other side websites like Facebook and Google+ show an notification icon even if nothing is there. |
Yeah, I was thinking about the other websites as well. But they definitely have more cases where notifications are shown. So as of now I would still say we should only show the icon when there are notifications. Once there are more cases where there can be notifications, we could show it permanently. We can revisit that for 9.2 or so. |
Since some time, the notifications icon shows by default in the top bar. Even though there are no notifications.
When there are no notifications, there is no need for the icon to show. @nickvergessen
The text was updated successfully, but these errors were encountered: