[wp-trac] [WordPress Trac] #40179: 404 for two emojis
    WordPress Trac 
    noreply at wordpress.org
       
    Fri Mar 17 10:55:24 UTC 2017
    
    
  
#40179: 404 for two emojis
-------------------------------+--------------------------
 Reporter:  Grzegorz.Janoszka  |       Owner:
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:
Component:  Emoji              |     Version:  4.7.3
 Severity:  normal             |  Resolution:  invalid
 Keywords:                     |     Focuses:  performance
-------------------------------+--------------------------
Comment (by SergeyBiryukov):
 Replying to [comment:2 Grzegorz.Janoszka]:
 > Please note that in /wp-includes/js/wp-emoji-release.min.js I see the
 following:
 > {{{
 > var
 p={base:"https://twemoji.maxcdn.com/2/",ext:".png",size:"72x72",className:"emoji",
 > }}}
 Right, that's the default value, but it's only used if no other value is
 specified via [source:tags/4.7.3/src/wp-includes/js/wp-
 emoji.js?marks=166#L155 settings.svgUrl]. WordPress does specify its own
 [source:tags/4.7.3/src/wp-includes/formatting.php?marks=5032#L5025 svgUrl
 value].
 > I actually dug into that and I found one plugin that was meant to speed
 up the website had a code:
 > {{{
 >   # remove <link rel='dns-prefetch' href='//s.w.org' /> from the head:
 >   add_filter('emoji_svg_url','__return_false');
 > }}}
 > Could it be that it actually changes the CDN from s.w.org to maxcdn?
 Yes, that's exactly what it does. If the goal is just to remove the `dns-
 prefetch` resource hint,
 [https://developer.wordpress.org/reference/hooks/wp_resource_hints/
 wp_resource_hints] filter should have been used for that.
 > And if yes - there seems to be some replication issues as some files are
 missing.
 For reference, the missing files are:
 * `slightly frowning face`:
 http://www.fileformat.info/info/unicode/char/1f641/index.htm
 * `slightly smiling face`:
 http://www.fileformat.info/info/unicode/char/1f642/index.htm
--
Ticket URL: <https://core.trac.wordpress.org/ticket/40179#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
    
    
More information about the wp-trac
mailing list