From f4a8ff415cd07fe6472eebe467bb897efecacd83 Mon Sep 17 00:00:00 2001 From: Steve Howell Date: Sat, 4 Aug 2018 12:44:16 +0000 Subject: [PATCH] minor: Fix comments for frequently used emojis. I think it's a known problem that we don't match on common aliases for our top N emojis, but I don't attempt to address that here. I just made the comments reflect the names we use in our current data structures. --- static/js/emoji_picker.js | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/static/js/emoji_picker.js b/static/js/emoji_picker.js index 3f06bf3434..d67cb521df 100644 --- a/static/js/emoji_picker.js +++ b/static/js/emoji_picker.js @@ -11,12 +11,17 @@ var APPROX_WIDTH = 255; // and composing a message with an emoji share a single widget, // implemented as the emoji_popover. exports.complete_emoji_catalog = []; + +// Unfortunately, this list does not currently match on +// alias names like party_popper, simple_smile, and +// hammer_and_wrench. But thumbs_up sorts to the top +// for some other reason. exports.frequently_used_emojis_list = [ - '1f44d', // thumbs_up - '1f389', // party_popper - '1f642', // simple_smile + '1f44d', // +1 + '1f389', // tada + '1f642', // slight_smile '2764', // heart - '1f6e0', // hammer_and_wrench + '1f6e0', // working_on_it '1f419', // octopus ]; var current_message_emoji_popover_elem;