Merge pull request #66 from owncloud/issue-65-hide-icon-on-no-notific…

…ations Hide the bell-icon when there are no notifications so we don't clutte…
owncloud · Feb 24, 2016 · 573758f · 573758f
2 parents 1bdafa8 + 734edd5
commit 573758f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/js/app.js b/js/app.js
@@ -36,7 +36,7 @@
 			// Go!
 
 			// Setup elements
-			this.$notifications = $('<div class="notifications"></div>');
+			this.$notifications = $('<div class="notifications hidden"></div>');
 			this.$button = $('<div class="notifications-button menutoggle"><img class="svg" alt="' + t('notifications', 'Notifications') + '" src="' + OC.imagePath('notifications', 'notifications') + '"></div>');
 			this.$container = $('<div class="notification-container"></div>');
 			var $wrapper = $('<div class="notification-wrapper"></div>');
@@ -297,6 +297,8 @@
 				.animate({opacity: 1}, 600)
 				.animate({opacity: 0.7}, 600);
 			this.$container.find('.emptycontent').addClass('hidden');
+
+			this.$notifications.removeClass('hidden');
 		},
 
 		/**
@@ -307,6 +309,8 @@
 			$('div.notifications-button').removeClass('hasNotifications');
 			$('div.notifications .emptycontent').removeClass('hidden');
 			this.$button.find('img').attr('src', OC.imagePath('notifications', 'notifications'));
+
+			this.$notifications.addClass('hidden');
 		},
 
 		/**