From 02bd5bedce0bf0b5389f9dddb19e5672f3a02d75 Mon Sep 17 00:00:00 2001 From: tejaskasetty Date: Mon, 20 Mar 2017 19:45:49 +0530 Subject: [PATCH] Fix duplicate emoji entries in compose emoji picker. We were incorrectly appending all the emoji into the emoji picker every time it was opened, rather than just once. Note by tabbott: Arguably this isn't the right fix, in that it might be better to just render the emoji picker once at the beginning. But this definitely fixes the bug. Fixes #3952. --- static/js/popovers.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/static/js/popovers.js b/static/js/popovers.js index 4f451f8fdd..6b120a5a1d 100644 --- a/static/js/popovers.js +++ b/static/js/popovers.js @@ -358,7 +358,7 @@ function render_emoji_popover() { realm_emoji: emoji.realm_emojis, }); }()); - + $('.emoji_popover').empty(); $('.emoji_popover').append(content); $('.drag').show();