Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#49861 closed enhancement (fixed)

Emoji javascript comment

Reported by: superpoincare's profile superpoincare Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.5 Priority: normal
Severity: normal Version: 5.4
Component: Build/Test Tools Keywords:
Focuses: javascript Cc:


The line

/*! This file is auto-generated */

appears in the front end of the emoji code in the head

It look unusual for it to appear in html.

Change History (2)

#1 @SergeyBiryukov
4 years ago

  • Component changed from General to Build/Test Tools
  • Milestone changed from Awaiting Review to 5.5

Thanks for the ticket!

Introduced in [46589] / #48424.

It seems like we could either:

  • Remove the banner from wp-emoji-release.min.js.
  • Or (preferably) use grunt-include-replace instead of grunt-includes for including the file, since it doesn't look like the latter can remove something from the file contents before including.

#2 @SergeyBiryukov
4 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 48096:

Build/Test Tools: Replace the banner text from wp-emoji-loader.min.js in formatting.php during build process.

This ensures that /*! This file is auto-generated */ comment is not included on front end as part of the inline Emoji detection script.

Follow-up to [41271], [46589].

Props superpoincare, SergeyBiryukov.
Fixes #49861.

Note: See TracTickets for help on using tickets.