WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#31893 closed defect (bug) (fixed)

include '//' in emoji url.

Reported by: tmatsuur Owned by: pento
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.2
Component: General Keywords:
Focuses: javascript Cc:

Description

Posts containing the Emoji I was displayed in Chrome on Windows 8.1.
Emoji have been rewritten to img element, were included '' in the URL of the img element.

Example: beer
http://s.w.org/images/core/emoji/72x72//1f37a.png

I think that the 72x72// is good in 72x72/.

head element[page source]:

window._wpemojiSettings = {"baseUrl":"http:\/\/s.w.org\/images\/core\/emoji\/72x72\/","ext":".png", ...

twemoji.parse[wp-emoji.js]: line130

return ''.concat( options.base, '/', icon, options.ext );

I think that it should do fixed in such a feeling.

return ''.concat( options.base, icon, options.ext );

Change History (2)

#1 @pento
4 years ago

  • Focuses javascript added
  • Milestone changed from Awaiting Review to 4.2
  • Owner set to pento
  • Status changed from new to assigned

#2 @pento
4 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 32024:

Emoji: Remove an errant / in Twemoji URLs.

Props tmatsuur.

Fixes #31893.

Note: See TracTickets for help on using tickets.