[wp-trac] [WordPress Trac] #31701: Emoji: Avoid enqueueing JS by default

WordPress Trac noreply at wordpress.org
Mon Mar 23 22:53:12 UTC 2015


#31701: Emoji: Avoid enqueueing JS by default
-----------------------------+--------------------------------------
 Reporter:  obenland         |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  4.2
Component:  General          |     Version:  trunk
 Severity:  major            |  Resolution:
 Keywords:  emoji has-patch  |     Focuses:  javascript, performance
-----------------------------+--------------------------------------

Comment (by azaozz):

 In 31701.10.diff:
 - Fix `readyState` and `readystatechange` (have to be on `document`).
 - Some cleanup:
  - No need to output the (cache busting) version string to JS and append
 it from there. Can append in PHP.
  - Renamed the "handle" for the concatenated file in an attempt to make
 the loading a bit more readable. May need better name :)
  - Shaved few more bytes off wp-emoji-loader.js.

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


More information about the wp-trac mailing list