Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#31938 closed defect (bug) (fixed)

Apply script_loader_src filters to emoji JavaScript files

Reported by:	peterwilsoncc	Owned by:	pento
Priority:	normal	Milestone:	4.2
Component:	General	Version:	4.2
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:	javascript

Description

Apply the script_loader_src filter to the emoji JavaScript files:

wp-includes/js/wp-emoji.js
wp-includes/js/twemoji.js
wp-includes/js/wp-emoji-loader.js

For use with CDNs, etc.

Please advise if this is not the best filter and I will supply a new patch.

Attachments (1)

31938.1.patch (988 bytes ) - added by peterwilsoncc 11 years ago.

Download all attachments as: .zip

Change History (5)

@peterwilsoncc
11 years ago

Attachment 31938.1.patch added

#1 @peterwilsoncc
11 years ago

Type defect (bug) → enhancement

#2 @pento
11 years ago

Keywords has-patch added
Milestone Awaiting Review → 4.2
Owner set to pento
Status new → assigned
Type enhancement → defect (bug)

Agreed, these URLs should be filtered.

#3 @pento
11 years ago

Resolution → fixed
Status assigned → closed

Emoji: The emoji JS files should be run through the script_loader_src filter, as they would be if they were registered scripts.

Props peterwilsoncc.

Fixes #31938.

This ticket was mentioned in Slack in #core by bartczyz. View the logs.

8 years ago

Note: See TracTickets for help on using tickets.

Download in other formats: