From 71999b28c407b2f7b53dbf3556e8890b27c2ca69 Mon Sep 17 00:00:00 2001 From: Moriel Schottlender Date: Wed, 9 Sep 2015 11:50:54 -0700 Subject: [PATCH] Only show 'mark all as read' AFTER there are notifications Hide the 'mark all read' button while we fetch notifications. We can't mark all as read before there are any actual unread notifications in the popup anyways. Change-Id: I38ace6a3f8b7898bdcd82ce650947f0c7ef319e2 --- modules/ooui/mw.echo.ui.NotificationBadgeWidget.js | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/ooui/mw.echo.ui.NotificationBadgeWidget.js b/modules/ooui/mw.echo.ui.NotificationBadgeWidget.js index 3eb073f1f..5aa31a3ae 100644 --- a/modules/ooui/mw.echo.ui.NotificationBadgeWidget.js +++ b/modules/ooui/mw.echo.ui.NotificationBadgeWidget.js @@ -210,6 +210,7 @@ } this.pushPending(); + this.markAllReadButton.toggle( false ); this.notificationsModel.fetchNotifications() .then( function ( idArray ) { // Clip again