From d28819f7957c6b032195df00711be87ba0da35eb Mon Sep 17 00:00:00 2001 From: David Lynch Date: Thu, 22 Sep 2016 11:23:48 -0500 Subject: [PATCH] ui.MWLinkAnnotationInspector: skip internal link checking if the value has changed If the value has changed since the event was fired, we're operating on old data, *and* another event will have been fired for that change anyway. So abort this check. Bug: T146306 Change-Id: Ia6682ec0218fd3fbc573582753d815a56963eb71 --- modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js b/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js index 973f500c07..6d9910126a 100644 --- a/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js +++ b/modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js @@ -182,6 +182,9 @@ ve.ui.MWLinkAnnotationInspector.prototype.onInternalLinkInputChange = function ( // schemas that use the full set of allowed characters, and we might get // more false positives by checking for them. // Note 3: We allow protocol-relative URIs here. + if ( this.internalAnnotationInput.getTextInputWidget().getValue() !== value ) { + return; + } if ( !this.allowProtocolInInternal && /^(?:[a-z][a-z0-9\$\-_@\.&!\*"'\(\),]*:)?\/\//i.test( value.trim() )