Henning Snater has uploaded a new change for review.

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

Change subject: sitelinkview: Improved pagesuggester performance
......................................................................

sitelinkview: Improved pagesuggester performance

(requires https://github.com/wmde/ValueView/pull/137)

Change-Id: Ibbff182edb7fa30bfc994a7a3e2802fb16d1148c
---
M composer.json
M lib/resources/jquery.wikibase/jquery.wikibase.sitelinkview.js
2 files changed, 7 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/28/180128/1

diff --git a/composer.json b/composer.json
index d6e4e38..a202db7 100644
--- a/composer.json
+++ b/composer.json
@@ -33,7 +33,7 @@
                "data-values/data-types": "~0.4.0",
                "data-values/serialization": "~1.0",
                "data-values/javascript": "~0.6.0",
-               "data-values/value-view": "~0.9.0",
+               "data-values/value-view": "~0.9.1",
 
                "wikibase/data-model": "~2.4",
                "wikibase/data-model-javascript": "~1.0",
diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.sitelinkview.js 
b/lib/resources/jquery.wikibase/jquery.wikibase.sitelinkview.js
index 0eaf996..6265667 100644
--- a/lib/resources/jquery.wikibase/jquery.wikibase.sitelinkview.js
+++ b/lib/resources/jquery.wikibase/jquery.wikibase.sitelinkview.js
@@ -217,10 +217,14 @@
                        dir: dir
                } ).pagesuggester( pageNameInputOptions );
 
+               var pagesuggester = $pageNameInput.data( 'pagesuggester' );
+
                $pageNameInput
                .on( 'pagesuggesterchange.' + this.widgetName, function( event 
) {
-                       self.setError();
-                       self._trigger( 'change' );
+                       if( !pagesuggester.isSearching() ) {
+                               self.setError();
+                               self._trigger( 'change' );
+                       }
                } );
 
                this.$link.find( '.wikibase-sitelinkview-page' 
).empty().append( $pageNameInput );
@@ -236,8 +240,6 @@
                        .siteselector( {
                                source: self.options.getAllowedSites
                        } );
-
-               var pagesuggester = $pageNameInput.data( 'pagesuggester' );
 
                // Disable initially and wait for valid site input:
                pagesuggester.disable();

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibbff182edb7fa30bfc994a7a3e2802fb16d1148c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Henning Snater <[email protected]>

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

Reply via email to