#35300 closed task (blessed) (fixed)
Clean up emoji compatibility code
| Reported by: | pento | Owned by: | pento |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.5 |
| Component: | Emoji | Version: | |
| Severity: | normal | Keywords: | emoji has-patch |
| Cc: | Focuses: |
Description
The code in wp-emoji-loader.js is fairly messy, and it's kind of annoying to have to remember to update several places when a new test is added (for example, we'll need a new test when Unicode 9 lands later in the year).
Let's make it easier to read, and fix how wp-emoji.js uses the settings.supports object, so that we don't have to keep updating it.
Attachments (1)
Change History (8)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
35300.diff makes it a bit nicer to work with - we only need to add a new
caseinbrowserSupportsEmoji(), then add that corresponding test to thetestsarray.