KartikMistry has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/390225 )
Change subject: Fix some more typos in comments
......................................................................
Fix some more typos in comments
Change-Id: I7e8ad7f8c62c0b08536420fd380169e28bf502ca
---
M modules/dashboard/ext.cx.suggestionlist.js
M modules/dm/translationunits/mw.cx.dm.TranslationUnit.js
M modules/mw.cx.TranslationController.js
M modules/tools/ext.cx.tools.formatter.js
M modules/tools/ext.cx.tools.link.js
M modules/tools/ext.cx.tools.manager.js
M modules/tools/ext.cx.tools.mtabuse.js
M modules/tools/ext.cx.tools.reference.js
M modules/tools/ext.cx.tools.template.editor.js
M modules/tools/mw.cx.tools.TranslationTool.js
M modules/translation/ext.cx.translation.loader.js
M modules/util/ext.cx.util.js
M modules/util/mw.cx.util.js
13 files changed, 16 insertions(+), 16 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ContentTranslation
refs/changes/25/390225/1
diff --git a/modules/dashboard/ext.cx.suggestionlist.js
b/modules/dashboard/ext.cx.suggestionlist.js
index 914bee6..eacb13d 100644
--- a/modules/dashboard/ext.cx.suggestionlist.js
+++ b/modules/dashboard/ext.cx.suggestionlist.js
@@ -858,14 +858,14 @@
* Sort the lists in their logical order to display.
*
* @param {Object[]} lists
- * @return {number[]} Orderded list ids.
+ * @return {number[]} Ordered list ids.
*/
CXSuggestionList.prototype.sortLists = function ( lists ) {
return Object.keys( lists ).sort( listCompare );
};
/**
- * Make the list expandable and collapsable.
+ * Make the list expandable and collapsible.
*
* @param {string} listId
*/
diff --git a/modules/dm/translationunits/mw.cx.dm.TranslationUnit.js
b/modules/dm/translationunits/mw.cx.dm.TranslationUnit.js
index 2ae450a..9f5959c 100644
--- a/modules/dm/translationunits/mw.cx.dm.TranslationUnit.js
+++ b/modules/dm/translationunits/mw.cx.dm.TranslationUnit.js
@@ -6,7 +6,7 @@
* translated to target language. The same paragraph can have a reference, a
link,
* a template and all of these sub elements are also translatable. That means,
* A translation unit can be a child of another translation unit. A MediaWiki
template
- * is a translation unit. At the same time, template paramters are also
translatable units.
+ * is a translation unit. At the same time, template parameters are also
translatable units.
* mw.cx.dm.TranslationUnit class abstracts the data manipulation over such a
translation
* unit during a translation.
*/
diff --git a/modules/mw.cx.TranslationController.js
b/modules/mw.cx.TranslationController.js
index 07b00ff..315d889 100644
--- a/modules/mw.cx.TranslationController.js
+++ b/modules/mw.cx.TranslationController.js
@@ -113,7 +113,7 @@
// This causes failCounter to increase because the in-flight
request fails.
// The new request we do below will either reset the fail
counter on success.
// If it does not succeed, the retry timer that was set by the
failed request
- // prevents further saves before the retry has completed
succesfully or given up.
+ // prevents further saves before the retry has completed
successfully or given up.
this.saveRequest.abort();
}
diff --git a/modules/tools/ext.cx.tools.formatter.js
b/modules/tools/ext.cx.tools.formatter.js
index 816e0b2..ec8326e 100644
--- a/modules/tools/ext.cx.tools.formatter.js
+++ b/modules/tools/ext.cx.tools.formatter.js
@@ -94,7 +94,7 @@
};
/**
- * Make the selection ttalic
+ * Make the selection italic
*/
FormatTool.prototype.italic = function () {
mw.cx.selection.restore( 'format' );
diff --git a/modules/tools/ext.cx.tools.link.js
b/modules/tools/ext.cx.tools.link.js
index 6375b13..7750d7f 100644
--- a/modules/tools/ext.cx.tools.link.js
+++ b/modules/tools/ext.cx.tools.link.js
@@ -721,7 +721,7 @@
}
if ( this.$link.hasClass( 'cx-target-link' ) ) {
- // Already adapted. Can be restrored from a saved
translation(draft)
+ // Already adapted. Can be restored from a saved
translation(draft)
this.adapted = true;
this.fetchLinkData();
return;
diff --git a/modules/tools/ext.cx.tools.manager.js
b/modules/tools/ext.cx.tools.manager.js
index b36d819..898d08b 100644
--- a/modules/tools/ext.cx.tools.manager.js
+++ b/modules/tools/ext.cx.tools.manager.js
@@ -174,7 +174,7 @@
/**
* Hide unrelated cards.
- * Find cards not having trigger events intersecton and hide them.
+ * Find cards not having trigger events intersection and hide them.
*
* @param {string} currentToolName current tool name
*/
diff --git a/modules/tools/ext.cx.tools.mtabuse.js
b/modules/tools/ext.cx.tools.mtabuse.js
index 5320034..6c94883 100644
--- a/modules/tools/ext.cx.tools.mtabuse.js
+++ b/modules/tools/ext.cx.tools.mtabuse.js
@@ -50,10 +50,10 @@
MTAbuseCard.prototype.isAbuse = function ( progress ) {
var mtPercentage = progress.mt / progress.any * 100 || 0;
- // Only use abuse detection when thresold is reached and there
are 5 or more
+ // Only use abuse detection when threshold is reached and there
are 5 or more
// sections with MT. Some times, even with less than 5
sections, the total
// translations can be a lot. So the condition then becomes:
Either the mtSectionCount
- // is geater than 5 or total translation is greater than 75%.
+ // is greater than 5 or total translation is greater than 75%.
// This also address the case of a source article having just 5
or less sections.
return mtPercentage > 75 &&
( progress.mtSectionsCount > 5 || progress.any * 100 >
75 );
diff --git a/modules/tools/ext.cx.tools.reference.js
b/modules/tools/ext.cx.tools.reference.js
index f3e452e..c6b6723 100644
--- a/modules/tools/ext.cx.tools.reference.js
+++ b/modules/tools/ext.cx.tools.reference.js
@@ -283,7 +283,7 @@
mwData.body = mwData.body || {};
if ( !mwData.body.id ) {
/*
- Every reference must have a data-mw.body with id
poiting to the item
+ Every reference must have a data-mw.body with id
pointing to the item
in References section. In general, we can just get copy
the data-mw
from the source reference. But there are cases it wont
be filled in source reference.
Example: When reference is reused more than once, the
second reference might not have
diff --git a/modules/tools/ext.cx.tools.template.editor.js
b/modules/tools/ext.cx.tools.template.editor.js
index 1876958..e2f5c05 100644
--- a/modules/tools/ext.cx.tools.template.editor.js
+++ b/modules/tools/ext.cx.tools.template.editor.js
@@ -386,7 +386,7 @@
};
/**
- * Edit handler for the parametr values
+ * Edit handler for the parameter values
*
* @param {Event} event Edit event
*/
diff --git a/modules/tools/mw.cx.tools.TranslationTool.js
b/modules/tools/mw.cx.tools.TranslationTool.js
index f974785..8d532d0 100644
--- a/modules/tools/mw.cx.tools.TranslationTool.js
+++ b/modules/tools/mw.cx.tools.TranslationTool.js
@@ -107,7 +107,7 @@
mw.cx.tools.TranslationTool.prototype.destroy = function () {
var card = this.getCard();
if ( card ) {
- // Remove will not work, untill this is attached to DOM
+ // Remove will not work, until this is attached to DOM
card.$element.remove();
card.$element = $( [] );
}
diff --git a/modules/translation/ext.cx.translation.loader.js
b/modules/translation/ext.cx.translation.loader.js
index d189c1c..9ffbfbe 100644
--- a/modules/translation/ext.cx.translation.loader.js
+++ b/modules/translation/ext.cx.translation.loader.js
@@ -106,11 +106,11 @@
mw.hook( 'mw.cx.translation.placeholders.ready' ).add(
function () {
if ( +self.translation.sourceRevisionId ===
+mw.cx.sourceRevision ||
// Very old drafts will have revision
id as 0. Consider them as original source for
- // the translation and use agressive
section restore algorithm
+ // the translation and use aggressive
section restore algorithm
+self.translation.sourceRevisionId === 0
) {
// Since we are using older revision,
the original revision used for translation,
- // use agressive section restore
algorithm.
+ // use aggressive section restore
algorithm.
self.originalRevision = true;
}
self.restore();
diff --git a/modules/util/ext.cx.util.js b/modules/util/ext.cx.util.js
index 1a719d4..efd4cff 100644
--- a/modules/util/ext.cx.util.js
+++ b/modules/util/ext.cx.util.js
@@ -96,7 +96,7 @@
/**
* Fix the ids of given element and its children so that it does not
conflict
- * with existing ids. This is done to html fragments recieved from
parsoid.
+ * with existing ids. This is done to html fragments received from
parsoid.
*
* @param {jQuery} $element The element whose ids to be fixed
* @param {string} prefix Prefix to use while fixing id
diff --git a/modules/util/mw.cx.util.js b/modules/util/mw.cx.util.js
index bd8a6ec..e693295 100644
--- a/modules/util/mw.cx.util.js
+++ b/modules/util/mw.cx.util.js
@@ -1,5 +1,5 @@
/**
- * A set of utilit methods for Content Translation
+ * A set of utility methods for Content Translation
*/
'use strict';
--
To view, visit https://gerrit.wikimedia.org/r/390225
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7e8ad7f8c62c0b08536420fd380169e28bf502ca
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContentTranslation
Gerrit-Branch: master
Gerrit-Owner: KartikMistry <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
