Adrian Heine has uploaded a new change for review.
https://gerrit.wikimedia.org/r/274397
Change subject: Do not eagerly initialize notification area
......................................................................
Do not eagerly initialize notification area
Change-Id: Icf7e969e1602f8681a0f1faea4458c9d8a195f9b
---
M view/resources/jquery/wikibase/jquery.wikibase.entitytermsview.js
1 file changed, 17 insertions(+), 14 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/97/274397/1
diff --git a/view/resources/jquery/wikibase/jquery.wikibase.entitytermsview.js
b/view/resources/jquery/wikibase/jquery.wikibase.entitytermsview.js
index 5f1ade2..32b0377 100644
--- a/view/resources/jquery/wikibase/jquery.wikibase.entitytermsview.js
+++ b/view/resources/jquery/wikibase/jquery.wikibase.entitytermsview.js
@@ -171,8 +171,7 @@
* @inheritdoc
*/
draw: function() {
- var self = this,
- deferred = $.Deferred();
+ var deferred = $.Deferred();
this.$entitytermsforlanguagelistview
= this.element.find(
'.wikibase-entitytermsforlanguagelistview' );
@@ -195,15 +194,9 @@
this._createEntitytermsforlanguagelistviewToggler();
}
- if ( !this._$notification ) {
- this.notification()
- .appendTo(
this._getEntitytermsforlanguagelistview().$header )
- .on( 'closeableupdate.' + this.widgetName,
function() {
- var sticknode = self.element.data(
'sticknode' );
- if ( sticknode ) {
- sticknode.refresh();
- }
- } );
+ if ( this._$notification ) {
+ // Reset notifications
+ this.notification();
}
return deferred.resolve().promise();
@@ -492,16 +485,26 @@
* @inheritdoc
*/
notification: function( $content, additionalCssClasses ) {
+ var self = this;
if ( !this._$notification ) {
var $closeable = $( '<div/>' ).closeable();
this._$notification = $( '<tr/>' ).append( $( '<td/>'
).append( $closeable ) );
this._$notification.data( 'closeable', $closeable.data(
'closeable' ) );
- }
+ this._$notification
+ .appendTo(
this._getEntitytermsforlanguagelistview().$header )
+ .on( 'closeableupdate.' + this.widgetName,
function() {
+ var sticknode =
self.element.data( 'sticknode' );
+ if ( sticknode ) {
+ sticknode.refresh();
+ }
+ } );
- var $headerTr =
this._getEntitytermsforlanguagelistview().$header.children( 'tr' ).first();
- this._$notification.children( 'td' ).attr( 'colspan',
$headerTr.children().length );
+ var $headerTr =
this._getEntitytermsforlanguagelistview().$header.children( 'tr' ).first();
+ this._$notification.children( 'td' ).attr( 'colspan',
$headerTr.children().length );
+
+ }
this._$notification.data( 'closeable' ).setContent( $content,
additionalCssClasses );
return this._$notification;
--
To view, visit https://gerrit.wikimedia.org/r/274397
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf7e969e1602f8681a0f1faea4458c9d8a195f9b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Adrian Heine <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
