Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Differential
D17244
Diff 41474
src/applications/macro/markup/PhabricatorEmojiRemarkupRule.php
Changeset View
We have to keep these intact, as-is, because the two lists are different. For example, if we make this change (removing this map), :+1: will break in all existing text.
This map basically has to be here forever since we shipped it, unless we're OK breaking a bunch of stuff or want to try to do a migration to replace all the :+1: in stuff in the database with the real emoji.
But you can just safely get rid of this now since the remarkup rule and the emoji typeahead no longer interact at all.