Hello Jack Phoenix, Sbisson, jenkins-bot, Siebrand, Mooeypoo,

I'd like you to do a code review.  Please visit

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

to review the following change.


Change subject: Revert "RCFilters: Change reset button title when it changes 
state"
......................................................................

Revert "RCFilters: Change reset button title when it changes state"

It makes no sense to add a tooltip to a button that already
has text, especially if the tooltip text is (almost)
the same as the button text.

This reverts commit 31047fb1bf03944158fe4f7da99686e7955a3b5c.

Change-Id: I2f9c819a85fc05f37c7c0c5acf2d72319bdb0d76
---
M languages/i18n/en.json
M languages/i18n/qqq.json
M resources/Resources.php
M 
resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterCapsuleMultiselectWidget.js
4 files changed, 0 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/69/343769/1

diff --git a/languages/i18n/en.json b/languages/i18n/en.json
index ea60fc8..ccfa91b 100644
--- a/languages/i18n/en.json
+++ b/languages/i18n/en.json
@@ -1366,7 +1366,6 @@
        "rcfilters-activefilters": "Active filters",
        "rcfilters-restore-default-filters": "Restore default filters",
        "rcfilters-clear-all-filters": "Clear all filters",
-       "rcfilters-clear-filters-to-defaults": "Set filters to defaults",
        "rcfilters-search-placeholder": "Filter recent changes (browse or start 
typing)",
        "rcfilters-invalid-filter": "Invalid filter",
        "rcfilters-empty-filter": "No active filters. All contributions are 
shown.",
diff --git a/languages/i18n/qqq.json b/languages/i18n/qqq.json
index 3116163..f123189 100644
--- a/languages/i18n/qqq.json
+++ b/languages/i18n/qqq.json
@@ -1553,7 +1553,6 @@
        "rcfilters-activefilters": "Title for the filters selection showing the 
active filters.",
        "rcfilters-restore-default-filters": "Label for the button that resets 
filters to defaults",
        "rcfilters-clear-all-filters": "Title for the button that clears all 
filters",
-       "rcfilters-clear-filters-to-defaults": "Title for the button that sets 
filters to default",
        "rcfilters-search-placeholder": "Placeholder for the filter search 
input.",
        "rcfilters-invalid-filter": "A label for an invalid filter.",
        "rcfilters-empty-filter": "Placeholder for the filter list when no 
filters were chosen.",
diff --git a/resources/Resources.php b/resources/Resources.php
index f57f8c8..392cdb3 100644
--- a/resources/Resources.php
+++ b/resources/Resources.php
@@ -1795,7 +1795,6 @@
                        'rcfilters-activefilters',
                        'rcfilters-restore-default-filters',
                        'rcfilters-clear-all-filters',
-                       'rcfilters-clear-filters-to-defaults',
                        'rcfilters-search-placeholder',
                        'rcfilters-invalid-filter',
                        'rcfilters-empty-filter',
diff --git 
a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterCapsuleMultiselectWidget.js
 
b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterCapsuleMultiselectWidget.js
index 3f461c7..944ebaa 100644
--- 
a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterCapsuleMultiselectWidget.js
+++ 
b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterCapsuleMultiselectWidget.js
@@ -183,11 +183,6 @@
                this.resetButton.setLabel(
                        currFiltersAreEmpty ? mw.msg( 
'rcfilters-restore-default-filters' ) : ''
                );
-               this.resetButton.setTitle(
-                       currFiltersAreEmpty ?
-                               mw.msg( 'rcfilters-clear-filters-to-defaults' ) 
:
-                               mw.msg( 'rcfilters-clear-all-filters' )
-               );
 
                this.resetButton.toggle( !hideResetButton );
                this.emptyFilterMessage.toggle( currFiltersAreEmpty );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2f9c819a85fc05f37c7c0c5acf2d72319bdb0d76
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
Gerrit-Reviewer: Jack Phoenix <[email protected]>
Gerrit-Reviewer: Mooeypoo <[email protected]>
Gerrit-Reviewer: Sbisson <[email protected]>
Gerrit-Reviewer: Siebrand <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to