Henning Snater has uploaded a new change for review.
https://gerrit.wikimedia.org/r/53375
Change subject: (bug 44543) Triggering snakviewchange when manipulating property
......................................................................
(bug 44543) Triggering snakviewchange when manipulating property
"snakviewchange" event is triggered when manipulating the snakview widget's
property input element now. External components (e.g. the toolbar) can now
react properly on when changing the property which might invalidate the snak.
Change-Id: I175c7a5d67a3fff125ed82237a049d2fc2711217
---
M lib/resources/jquery.wikibase/jquery.wikibase.snaklistview.js
M lib/resources/jquery.wikibase/jquery.wikibase.snakview/snakview.js
2 files changed, 4 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/75/53375/1
diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.snaklistview.js
b/lib/resources/jquery.wikibase/jquery.wikibase.snaklistview.js
index 88a4d60..f5ed6b5 100644
--- a/lib/resources/jquery.wikibase/jquery.wikibase.snaklistview.js
+++ b/lib/resources/jquery.wikibase/jquery.wikibase.snaklistview.js
@@ -427,7 +427,7 @@
$.each( this._listview.items(), function( i, item ) {
var snakview = self._lia.liInstance( $( item ) );
- if ( !snakview.isValid() ) {
+ if ( !snakview.isValid() || !snakview.snak() ) {
isValid = false;
return false;
} else {
diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.snakview/snakview.js
b/lib/resources/jquery.wikibase/jquery.wikibase.snakview/snakview.js
index cfd542a..d270b5a 100644
--- a/lib/resources/jquery.wikibase/jquery.wikibase.snakview/snakview.js
+++ b/lib/resources/jquery.wikibase/jquery.wikibase.snakview/snakview.js
@@ -196,6 +196,7 @@
// remove out-dated variations
if( self._variation ) {
self.propertyId( null );
+ self._trigger( 'change' );
}
} )
.on( 'entityselectorselect', function( e, ui ) {
@@ -228,6 +229,8 @@
self.propertyId( entityId );
+ self._trigger( 'change' );
+
// Since it takes a while for the value view to
gather its data from the API,
// the property might not be valid anymore
aborting the rendering of the value
// view.
--
To view, visit https://gerrit.wikimedia.org/r/53375
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I175c7a5d67a3fff125ed82237a049d2fc2711217
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
