From 1553a1a520c8c716acac80a0ebd3b78fbba397ec Mon Sep 17 00:00:00 2001 From: Moriel Schottlender Date: Thu, 28 Jul 2016 15:47:42 -0700 Subject: [PATCH] Remove 'mark as unread' from dotdotdot menu in items Bug: T140517 Change-Id: Ie6c0f003239d4a683a1651df9470074668faa4eb --- modules/ui/mw.echo.ui.NotificationItemWidget.js | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/modules/ui/mw.echo.ui.NotificationItemWidget.js b/modules/ui/mw.echo.ui.NotificationItemWidget.js index 4cbc94284..f5bbabfbb 100644 --- a/modules/ui/mw.echo.ui.NotificationItemWidget.js +++ b/modules/ui/mw.echo.ui.NotificationItemWidget.js @@ -150,13 +150,6 @@ } } - // Add a "mark as read" secondary action - this.toggleReadSecondaryButton = new mw.echo.ui.MenuItemWidget( { - data: 'toggleRead', - prioritized: false - } ); - this.menuPopupButtonWidget.getMenu().addItems( [ this.toggleReadSecondaryButton ] ); - if ( this.bundle ) { // In a bundle, we have table layout, so the icon is // inserted into the content, and the 'mark as read' @@ -321,16 +314,6 @@ showMarkAsRead = showMarkAsRead !== undefined ? showMarkAsRead : !this.model.isRead(); this.markAsReadButton.toggleState( showMarkAsRead ); - - if ( showMarkAsRead ) { - // Mark read - this.toggleReadSecondaryButton.setLabel( mw.msg( 'echo-notification-markasread' ) ); - this.toggleReadSecondaryButton.setIcon( 'check' ); - } else { - // Mark unread - this.toggleReadSecondaryButton.setLabel( mw.msg( 'echo-notification-markasunread' ) ); - this.toggleReadSecondaryButton.setIcon( 'sun' ); - } this.menuPopupButtonWidget.toggle( !this.menuPopupButtonWidget.getMenu().isEmpty() ); };