jenkins-bot has submitted this change and it was merged.

Change subject: Hide "remove" button when adding a new statement
......................................................................


Hide "remove" button when adding a new statement

(bug 56096)
"Remove" button is not initialized anymore when adding a new statement.

Change-Id: If020dd22930f4dd308417ce2eb741cad77351948
---
M lib/resources/jquery.wikibase/toolbar/toolbarcontroller.js
1 file changed, 3 insertions(+), 8 deletions(-)

Approvals:
  Aude: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/lib/resources/jquery.wikibase/toolbar/toolbarcontroller.js 
b/lib/resources/jquery.wikibase/toolbar/toolbarcontroller.js
index 30d6416..376d2a2 100644
--- a/lib/resources/jquery.wikibase/toolbar/toolbarcontroller.js
+++ b/lib/resources/jquery.wikibase/toolbar/toolbarcontroller.js
@@ -147,19 +147,14 @@
                                $.each( self.options[type], function( j, 
definitionId ) {
                                        var def = 
$.wikibase.toolbarcontroller.definition( type, definitionId ),
                                                eventPrefix = def.eventPrefix
-                                                       || ( def.widget ? 
def.widget.prototype.widgetEventPrefix : '' ),
-                                               baseClass = def.baseClass
-                                                       || ( def.widget ? 
def.widget.prototype.widgetBaseClass : null );
+                                                       || ( def.widget ? 
def.widget.prototype.widgetEventPrefix : '' );
 
                                        // Listen to widget's native "create" 
event in order to initialize toolbars
                                        // corresponding to the widget just 
instantiated.
                                        self.element.on( eventPrefix + 
'create.' + self.widgetName, function( event ) {
                                                var $target = $( event.target ),
-                                                       isPending = baseClass
-                                                               && (
-                                                                       
$target.hasClass( baseClass + '-new' )
-                                                                       || 
$target.find( baseClass + '-new' ).length > 0
-                                                               );
+                                                       isWidget = !!def.widget,
+                                                       isPending = isWidget && 
!$target.data( def.widget.name ).value();
 
                                                if ( type === 'addtoolbar' ) {
                                                        // Initialize toolbars 
that are not initialized already:

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If020dd22930f4dd308417ce2eb741cad77351948
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: mw1.23-wmf1
Gerrit-Owner: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>
Gerrit-Reviewer: Aude <aude.w...@gmail.com>
Gerrit-Reviewer: Henning Snater <henning.sna...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to