Santhosh has uploaded a new change for review.

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


Change subject: (bug 45494) Make "translated" the default filter for 
proofreading
......................................................................

(bug 45494) Make "translated" the default filter for proofreading

Change-Id: Ifef5276a3a99fcd0a69abf2b173f332da93adcee
---
M resources/js/ext.translate.messagetable.js
1 file changed, 11 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Translate 
refs/changes/44/51344/1

diff --git a/resources/js/ext.translate.messagetable.js 
b/resources/js/ext.translate.messagetable.js
index fac3c1a..e8b3cc7 100644
--- a/resources/js/ext.translate.messagetable.js
+++ b/resources/js/ext.translate.messagetable.js
@@ -364,7 +364,8 @@
                 * @param {string} mode The message table mode - proofread or 
translate
                 */
                switchMode: function ( mode ) {
-                       var messageTable = this;
+                       var messageTable = this,
+                               filter = messageTable.$loader.data( 'filter' );;
 
                        messageTable.$actionBar.find( '.down').removeClass( 
'down' );
                        if ( mode === 'translate' ) {
@@ -385,17 +386,19 @@
 
                        if ( messageTable.mode === 'proofread' ) {
                                $( '.tux-message-selector > 
.tux-tab-untranslated' ).addClass( 'hide' );
+                               // Fix the filter if it is untranslated. 
Untranslated does not make sense
+                               // for proofread mode. Keep the filter if it is 
not 'untranslated'
+                               if ( filter.indexOf( '!translated' ) >= 0 )  {
+                                       messageTable.messages = [];
+                                       mw.translate.changeFilter( 'translated' 
);
+                                       $( '.tux-message-selector > 
.tux-tab-translated' ).addClass( 'selected' );
+                               }
                        } else {
                                $( '.tux-message-selector > 
.tux-tab-untranslated' ).removeClass( 'hide' );
                        }
 
-                       $.each( messageTable.messages, function ( index, 
message ) {
-                               if ( messageTable.mode === 'translate' ) {
-                                       messageTable.addTranslate( message );
-                               }
-                               if ( messageTable.mode === 'proofread' ) {
-                                       messageTable.addProofread( message );
-                               }
+                       $.each( messageTable.messages, function ( inddex, 
message ) {
+                               messageTable.add( message );
                        } );
                },
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifef5276a3a99fcd0a69abf2b173f332da93adcee
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Santhosh <[email protected]>

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

Reply via email to