jenkins-bot has submitted this change and it was merged. Change subject: Trim whitespace from link targets ......................................................................
Trim whitespace from link targets We don't need stuff like href="%09http://example.com ". Bug: T94498 Change-Id: I4df99d2c12d67275b999f0eb1234bd7cd44fe10e --- M src/ui/widgets/ve.ui.LinkTargetInputWidget.js 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Alex Monk: Looks good to me, approved jenkins-bot: Verified diff --git a/src/ui/widgets/ve.ui.LinkTargetInputWidget.js b/src/ui/widgets/ve.ui.LinkTargetInputWidget.js index b6fcd57..e26a5e5 100644 --- a/src/ui/widgets/ve.ui.LinkTargetInputWidget.js +++ b/src/ui/widgets/ve.ui.LinkTargetInputWidget.js @@ -16,7 +16,10 @@ ve.ui.LinkTargetInputWidget = function VeUiLinkTargetInputWidget( config ) { // Parent constructor OO.ui.TextInputWidget.call( this, $.extend( { - validate: /^(https?:\/\/)?[\w-]+(\.[\w-]+)+\.?(:\d+)?(\/\S*)?/gi + validate: /^(https?:\/\/)?[\w-]+(\.[\w-]+)+\.?(:\d+)?(\/\S*)?/gi, + inputFilter: function ( target ) { + return target.trim(); + } }, config ) ); // Properties -- To view, visit https://gerrit.wikimedia.org/r/200804 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4df99d2c12d67275b999f0eb1234bd7cd44fe10e Gerrit-PatchSet: 1 Gerrit-Project: VisualEditor/VisualEditor Gerrit-Branch: master Gerrit-Owner: Catrope <roan.katt...@gmail.com> Gerrit-Reviewer: Alex Monk <kren...@gmail.com> Gerrit-Reviewer: Esanders <esand...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits