From bb35d9fc07b795fbd7d51b00ebd40c1d2781e0d5 Mon Sep 17 00:00:00 2001
From: Joffrey JAFFEUX <j.jaffeux@gmail.com>
Date: Fri, 21 Jul 2017 21:42:19 +0200
Subject: [PATCH] FIX: do not restore scroll on successive click on the same
 first emoji

---
 .../javascripts/discourse/components/emoji-picker.js.es6    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/discourse/components/emoji-picker.js.es6 b/app/assets/javascripts/discourse/components/emoji-picker.js.es6
index fdafa00377b..1c42006b557 100644
--- a/app/assets/javascripts/discourse/components/emoji-picker.js.es6
+++ b/app/assets/javascripts/discourse/components/emoji-picker.js.es6
@@ -90,6 +90,7 @@ export default Ember.Component.extend({
     const $recentSection = $list.find(".section[data-section='recent']");
     const $recentSectionGroup = $recentSection.find(".section-group");
     const $recentCategory = $picker.find(".category-icon button[data-section='recent']").parent();
+    let persistScrollPosition = !$recentCategory.is(':visible') ? true : false;
 
     // we set height to 0 to avoid it being taken into account for scroll position
     if(_.isEmpty(this.get("recentEmojis"))) {
@@ -105,11 +106,12 @@ export default Ember.Component.extend({
     });
     const template = findRawTemplate("emoji-picker-recent")({recentEmojis});
     $recentSectionGroup.html(template);
-    this._bindHover($recentSectionGroup);
 
-    if(this.get("recentEmojis").length === 1) {
+    if(persistScrollPosition) {
       $list.scrollTop(previousScrollTop + $recentSection.outerHeight());
     }
+
+    this._bindHover($recentSectionGroup);
   },
 
   close() {