[wp-trac] [WordPress Trac] #34640: Twemoji fallback doesn't kick in on subsequent DOM mutations
WordPress Trac
noreply at wordpress.org
Fri Nov 13 00:30:39 UTC 2015
#34640: Twemoji fallback doesn't kick in on subsequent DOM mutations
-----------------------------------+-------------------------
Reporter: rmccue | Owner: pento
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 4.4
Component: General | Version: 4.2
Severity: normal | Resolution:
Keywords: 2nd-opinion has-patch | Focuses: javascript
-----------------------------------+-------------------------
Comment (by pento):
@iseulde: Good idea, I've revised the patch so that `onerror` is now an
option - we can easily contribute that back upstream.
Unfortunately, we need to keep several of the checks in that conditional
(need to make sure `removedNodes[0]` will be defined, and that `nodeName
=== 'IMG'`, because text nodes don't have the `getAttribute()` method. I
just left the conditional the same in [attachment:34640.2.diff], but if
you're comfortable removing some of the checks, I'm cool with that.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/34640#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list