From a22aeafc069d1b96065372dfd156131ca31fe37d Mon Sep 17 00:00:00 2001 From: Ed Sanders Date: Sat, 16 Aug 2014 16:27:30 +0100 Subject: [PATCH] Remove span-wrapping of link='' images This hack is no longer neccessary now that we no longer use shields. Change-Id: I2194b16273b1ebe6e8d9360957cc7eab856dae13 --- modules/ve-mw/ce/nodes/ve.ce.MWInlineImageNode.js | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/modules/ve-mw/ce/nodes/ve.ce.MWInlineImageNode.js b/modules/ve-mw/ce/nodes/ve.ce.MWInlineImageNode.js index fc5af5b1ed..ab91e56d30 100644 --- a/modules/ve-mw/ce/nodes/ve.ce.MWInlineImageNode.js +++ b/modules/ve-mw/ce/nodes/ve.ce.MWInlineImageNode.js @@ -25,11 +25,7 @@ ve.ce.MWInlineImageNode = function VeCeMWInlineImageNode( model, config ) { this.$element = this.$( '' ).addClass( 'image' ); this.$image = this.$( '' ).appendTo( this.$element ); } else { - // For inline images that are not linked (empty linkto=) we intentionally don't match output - // of MW Parser, instead we wrap those images in span so selection and hover (based on - // shields) can work well. It might change in the future when we improve our selection. - this.$element = this.$( '' ); - this.$image = this.$( '' ).appendTo( this.$element ); + this.$element = this.$image = this.$( '' ).appendTo( this.$element ); } // Mixin constructors