#35300 closed task (blessed) (fixed)
Clean up emoji compatibility code
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.5 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Emoji | Keywords: | emoji has-patch |
| Focuses: | Cc: |
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.
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.