Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/344205 )

Change subject: Connect onModelUpdate to the right view node
......................................................................

Connect onModelUpdate to the right view node

Pretty sure this event does nothing for block images, but
if we are going to bind, bind the the ce caption node, not
the ce image node.

Change-Id: I02b5500f381e932efba87d61ccdf668cda377757
---
M modules/ve-mw/ce/nodes/ve.ce.MWBlockImageNode.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/05/344205/1

diff --git a/modules/ve-mw/ce/nodes/ve.ce.MWBlockImageNode.js 
b/modules/ve-mw/ce/nodes/ve.ce.MWBlockImageNode.js
index 251418c..4a1be91 100644
--- a/modules/ve-mw/ce/nodes/ve.ce.MWBlockImageNode.js
+++ b/modules/ve-mw/ce/nodes/ve.ce.MWBlockImageNode.js
@@ -139,7 +139,7 @@
                if ( !this.$caption ) {
                        model = this.model.children[ 0 ];
                        view = ve.ce.nodeFactory.create( model.getType(), model 
);
-                       model.connect( this, { update: 'onModelUpdate' } );
+                       model.connect( view, { update: 'onModelUpdate' } );
                        this.children.push( view );
                        view.attach( this );
                        if ( this.live !== view.isLive() ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I02b5500f381e932efba87d61ccdf668cda377757
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>

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

Reply via email to