Edit insertHTML fix: don't purge user-generated SPANs

guardian / scribe

DEPRECATED: A rich text editor framework for the web platform

http://guardian.github.io/scribe/

Apache License 2.0

3.51k stars 245 forks source link

Edit insertHTML fix: don't purge user-generated SPANs #385

Closed christopherliu closed 9 years ago

christopherliu commented 9 years ago

A while back, this was written in insert-html.js: "FIXME: what if the user actually wants to use SPANs? This could cause conflicts." Turns out that user was me. This updated patch is more targeted, removing only SPANs that are truly superfluous. In addition, it also covers more than one SPAN, where the original code would only cover the first child SPAN.

This was originally #339

christopherliu commented 9 years ago

Thanks!