[wp-trac] [WordPress Trac] #32305: Emoji script is producing errors on pages with SVG content

WordPress Trac noreply at wordpress.org
Fri May 8 18:25:20 UTC 2015


#32305: Emoji script is producing errors on pages with SVG content
--------------------------+-------------------------
 Reporter:  martynasma    |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  4.2.3
Component:  Formatting    |     Version:  4.2.2
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  javascript
--------------------------+-------------------------

Comment (by martynasma):

 Replying to [comment:1 azaozz]:
 > Looks like this can be fixed by doing:
 >
 > {{{
 > if ( ! node || ( node.className && typeof node.className === 'string' &&
 node.className.indexOf( 'wp-exclude-emoji' ) !== -1 ) ) {
 >    continue;
 > }
 > }}}
 >
 > However, seems we shouldn't be looking inside SVGs in the first place.
 Not sure what will happen if we replace emoji chars inside SVGs.

 Yes, I suppose trying to modify SVG content doesn't make much sense and is
 quite dangerous, since it's not HTML.

 I hope you can find a way to exclude SVG from emoji application.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32305#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list