[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