wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-11-24 22:35:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

ve.dm.MWTransclusionModel: Remove workaround for T135127

Browse source 
The bug this works around seems to be fixed.

Change-Id: I62837ad795ee6bd35663b163f6de76572e1a6594
This commit is contained in:
Bartosz Dziewoński 2018-09-05 18:56:52 +02:00
parent 79728eedc7
commit 18f8697703
1 changed files with 2 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
modules/ve-mw/dm/models/ve.dm.MWTransclusionModel.js
View file

@ -70,7 +70,6 @@
function insertNode( isInline, generatedContents ) {
var hash, store, nodeClass,
type = isInline ? baseNodeClass.static.inlineType : baseNodeClass.static.blockType,
range = surfaceFragment.getSelection().getCoveringRange(),
data = [
{
type: type,
@ -90,16 +89,8 @@
store.hash( generatedContents, hash );
}
if ( range.isCollapsed() ) {
surfaceFragment.insertContent( data );
} else {
// Generate a replacement transaction instead of using surfaceFragment.insertContent
// (which generates a removal and insertion) as blanking a reference triggers T135127.
// TODO: Once T135127 is fixed, revert to using surfaceFragment.insert.
surfaceFragment.getSurface().change(
ve.dm.TransactionBuilder.static.newFromReplacement( surfaceFragment.getDocument(), range, data )
);
}
surfaceFragment.insertContent( data );
deferred.resolve();
}