Sbisson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/283746

Change subject: Add tooltips for 'mark as read' and 'more options'
......................................................................

Add tooltips for 'mark as read' and 'more options'

Bug: T130831
Change-Id: I1533d20019157c05f9354a60661512a33920c336
---
M Resources.php
M i18n/en.json
M i18n/qqq.json
M modules/ooui/mw.echo.ui.NotificationItemWidget.js
4 files changed, 9 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/46/283746/1

diff --git a/Resources.php b/Resources.php
index feea1c2..df71cbc 100644
--- a/Resources.php
+++ b/Resources.php
@@ -133,6 +133,8 @@
                        "notification-timestamp-ago-years",
                        'echo-notification-markasread',
                        'echo-notification-markasunread',
+                       'echo-notification-markasread-tooltip',
+                       'echo-notification-more-options-tooltip',
                        'echo-notification-alert-text-only',
                        'echo-notification-message-text-only',
                        'echo-email-batch-bullet',
diff --git a/i18n/en.json b/i18n/en.json
index 258e31e..ae046ea 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -96,6 +96,8 @@
        "echo-notification-popup-loginrequired": "Please log in to view your 
notifications.",
        "echo-notification-markasread": "Mark as read",
        "echo-notification-markasunread": "Mark as unread",
+       "echo-notification-markasread-tooltip": "Mark as read",
+       "echo-notification-more-options-tooltip": "More options",
        "notification-link-text-expand-all": "View all",
        "notification-link-text-expand-alert-count": "View {{PLURAL:$1|$1 
alert|$1 alerts}}",
        "notification-link-text-expand-message-count": "View {{PLURAL:$1|$1 
message|$1 messages}}",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index ecfd82b..7c3cb37 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -87,6 +87,8 @@
        "echo-notification-popup-loginrequired": "Message that displays when an 
anonymous user attempts to view notifications in their popup after a session 
expired.",
        "echo-notification-markasread": "Label for the button to mark the 
notification as read.",
        "echo-notification-markasunread": "Label for the button to mark the 
notification as unread.",
+       "echo-notification-markasread-tooltip": "Tooltip for the button to mark 
the notification as read.",
+       "echo-notification-more-options-tooltip": "Tooltip for the button to 
show the hidden secondary actions.",
        "notification-link-text-expand-all": "Label for the button that expands 
a bundled notification.\n{{Identical|View all}}",
        "notification-link-text-expand-alert-count": "Label for the button that 
expands a bundled alert notification.\n\nParameters:\n* $1 - The count of 
messages that the bundle holds.\n{{Identical|View alert}}",
        "notification-link-text-expand-message-count": "Label for the button 
that expands a bundled message notification.\n\nParameters:\n* $1 - The count 
of messages that the bundle holds.\n{{Identical|View message}}",
diff --git a/modules/ooui/mw.echo.ui.NotificationItemWidget.js 
b/modules/ooui/mw.echo.ui.NotificationItemWidget.js
index 81552ba..26621cf 100644
--- a/modules/ooui/mw.echo.ui.NotificationItemWidget.js
+++ b/modules/ooui/mw.echo.ui.NotificationItemWidget.js
@@ -32,10 +32,11 @@
                this.$actions = $( '<div>' )
                        .addClass( 
'mw-echo-ui-notificationItemWidget-content-actions' );
 
-               // Mark unread
+               // Mark as read
                this.markAsReadButton = new OO.ui.ButtonWidget( {
                        icon: 'close',
                        framed: false,
+                       title: mw.msg( 'echo-notification-markasread-tooltip' ),
                        classes: [ 
'mw-echo-ui-notificationItemWidget-markAsReadButton' ]
                } );
 
@@ -76,6 +77,7 @@
                        icon: 'ellipsis',
                        $overlay: this.$overlay,
                        menuWidth: 200,
+                       title: mw.msg( 'echo-notification-more-options-tooltip' 
),
                        classes: [ 
'mw-echo-ui-notificationItemWidget-content-actions-menu' ]
                } );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/283746
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1533d20019157c05f9354a60661512a33920c336
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Sbisson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to