jenkins-bot has submitted this change and it was merged. Change subject: Cleanup page class names and comments ......................................................................
Cleanup page class names and comments Change-Id: I739b5d9b295d09b86104133a8f4eb30070d664f2 --- M modules/ve-mw/ui/pages/ve.ui.MWParameterPage.js M modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js M modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js 3 files changed, 3 insertions(+), 4 deletions(-) Approvals: Catrope: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/ve-mw/ui/pages/ve.ui.MWParameterPage.js b/modules/ve-mw/ui/pages/ve.ui.MWParameterPage.js index 65218e5..c638257 100644 --- a/modules/ve-mw/ui/pages/ve.ui.MWParameterPage.js +++ b/modules/ve-mw/ui/pages/ve.ui.MWParameterPage.js @@ -16,7 +16,7 @@ * @param {string} name Unique symbolic name of page * @param {Object} [config] Configuration options */ -ve.ui.MWParameterPage = function VeUiMWParameter( parameter, name, config ) { +ve.ui.MWParameterPage = function VeUiMWParameterPage( parameter, name, config ) { var paramName = parameter.getName(); // Parent constructor @@ -48,7 +48,6 @@ } ) .connect( this, { 'click': 'onRemoveButtonClick' } ); - // TODO: Use spec.required // TODO: Use spec.deprecation // TODO: Use spec.type diff --git a/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js b/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js index a54dbeb..8b7d5c2 100644 --- a/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js +++ b/modules/ve-mw/ui/pages/ve.ui.MWTemplatePlaceholderPage.js @@ -16,7 +16,7 @@ * @param {string} name Unique symbolic name of page * @param {Object} [config] Configuration options */ -ve.ui.MWTemplatePlaceholderPage = function VeUiMWTemplatePlaceholder( placeholder, name, config ) { +ve.ui.MWTemplatePlaceholderPage = function VeUiMWTemplatePlaceholderPage( placeholder, name, config ) { // Parent constructor OO.ui.PageLayout.call( this, name, config ); diff --git a/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js b/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js index 1c697b7..dce6262 100644 --- a/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js +++ b/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js @@ -16,7 +16,7 @@ * @param {string} name Unique symbolic name of page * @param {Object} [config] Configuration options */ -ve.ui.MWTransclusionContentPage = function VeUiMWTransclusionContent( content, name, config ) { +ve.ui.MWTransclusionContentPage = function VeUiMWTransclusionContentPage( content, name, config ) { // Parent constructor OO.ui.PageLayout.call( this, name, config ); -- To view, visit https://gerrit.wikimedia.org/r/116009 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I739b5d9b295d09b86104133a8f4eb30070d664f2 Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/extensions/VisualEditor Gerrit-Branch: master Gerrit-Owner: Trevor Parscal <tpars...@wikimedia.org> Gerrit-Reviewer: Catrope <roan.katt...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits