Merge pull request #9 from nextcloud/theming-invert-icon

Switch notification icon to dark if theming app is using bright colors
nextcloud · Aug 10, 2016 · a918c4e · a918c4e
2 parents cdbbd16 + 0b0340b
commit a918c4e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/img/notifications-new-dark.svg b/img/notifications-new-dark.svg
diff --git a/js/app.js b/js/app.js
@@ -289,8 +289,14 @@
 		 */
 		_onHaveNotifications: function() {
 			// Add the button, title, etc
+			var icon;
+			if (OCA.Theming && OCA.Theming.inverted) {
+				icon = 'notifications-new-dark';
+			} else {
+				icon = 'notifications-new';
+			}
 			this.$button.addClass('hasNotifications');
-			this.$button.find('img').attr('src', OC.imagePath('notifications', 'notifications-new'))
+			this.$button.find('img').attr('src', OC.imagePath('notifications', icon))
 				.animate({opacity: 0.5}, 600)
 				.animate({opacity: 1}, 600)
 				.animate({opacity: 0.5}, 600)