strike / del isn't replaced correctly in inwardHtml

[xinha.20.artus83@…]

In both functions inwardHTML and outwardHTML <strike> is replaced by <del>. I think in the function inwardHTML it should be the other way around.

So we should replace there:

    html = html.replace(/<(\/?)strike(\s|>|\/)/ig, "<$1del$2");

with:

    html = html.replace(/<(\/?)del(\s|>|\/)/ig, "<$1strike$2");

[niko]

why is it replaced anyway?

[xinha.20.artus83@…]

I think it is done, because IE creates <del> tags, but these are not recognized by the Mozilla editor component. So Mozilla user wouldn't be able to remove a style applied by an IE user.

[niko]

commited in changeset:342

[xinha.20.artus83@…]

I was wrong, with the reason it is done. IE creates <strike> tags to. In outwardHtml the tags replaced in both IE and Mozilla, but inwardHtml replaces them only in Mozilla. This should be done consistently, since the way it is at the moment, you can't remove the strike style in IE after viewing the source.

[ray]

I think the replacement maybe because <strike> is deprecated, and anyway logical markup nowadays is prefered to optical

[gogo]

Fixed - changeset:393