jenkins-bot has submitted this change and it was merged.
Change subject: ve.ce.BranchNode: Use DOM rather than jQuery in #onSplice
......................................................................
ve.ce.BranchNode: Use DOM rather than jQuery in #onSplice
Change-Id: I75337048e9ae81c95333da5c0277dbf7dc562b1c
---
M modules/ve/ce/ve.ce.BranchNode.js
1 file changed, 17 insertions(+), 3 deletions(-)
Approvals:
Krinkle: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ve/ce/ve.ce.BranchNode.js
b/modules/ve/ce/ve.ce.BranchNode.js
index 6704aa4..c6d6050 100644
--- a/modules/ve/ce/ve.ce.BranchNode.js
+++ b/modules/ve/ce/ve.ce.BranchNode.js
@@ -149,10 +149,14 @@
* @param {ve.dm.BranchNode...} [nodes] Variadic list of nodes to insert
*/
ve.ce.BranchNode.prototype.onSplice = function ( index ) {
- var i,
+ var i, j,
length,
args = Array.prototype.slice.call( arguments ),
$anchor,
+ afterAnchor,
+ node,
+ parentNode,
+ firstChild,
removals;
// Convert models to views and attach them to this node
if ( args.length >= 3 ) {
@@ -176,9 +180,19 @@
for ( i = args.length - 1; i >= 2; i-- ) {
args[i].attach( this );
if ( index ) {
- $anchor.after( args[i].$ );
+ // DOM equivalent of $anchor.after( args[i].$ );
+ afterAnchor = $anchor[0].nextSibling;
+ parentNode = $anchor[0].parentNode;
+ for ( j = 0, length = args[i].$.length; j <
length; j++ ) {
+ parentNode.insertBefore( args[i].$[j],
afterAnchor );
+ }
} else {
- this.$.prepend( args[i].$ );
+ // DOM equivalent of this.$.prepend( args[j].$
);
+ node = this.$[0];
+ firstChild = node.firstChild;
+ for ( j = args[i].$.length - 1; j >= 0; j-- ) {
+ node.insertBefore( args[i].$[j],
firstChild );
+ }
}
if ( this.live !== args[i].isLive() ) {
args[i].setLive( this.live );
--
To view, visit https://gerrit.wikimedia.org/r/69763
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I75337048e9ae81c95333da5c0277dbf7dc562b1c
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
