#35300 closed task (blessed) (fixed)
Clean up emoji compatibility code
Reported by: | pento | Owned by: | pento |
---|---|---|---|
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
case
inbrowserSupportsEmoji()
, then add that corresponding test to thetests
array.