From ae8723451af2ae75fd2bd46e9791dc069d56905c Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@matrix.org>
Date: Fri, 19 Oct 2018 14:27:36 +0200
Subject: [PATCH] remove onShowMoreRooms, as room tiles & scrollbar rendered at
 same level

---
 src/components/structures/RoomSubList.js |  4 ----
 src/components/views/rooms/RoomList.js   | 11 -----------
 2 files changed, 15 deletions(-)

diff --git a/src/components/structures/RoomSubList.js b/src/components/structures/RoomSubList.js
index a6117b1a00..5833a8b505 100644
--- a/src/components/structures/RoomSubList.js
+++ b/src/components/structures/RoomSubList.js
@@ -52,7 +52,6 @@ const RoomSubList = React.createClass({
         onHeaderClick: PropTypes.func,
         alwaysShowHeader: PropTypes.bool,
         incomingCall: PropTypes.object,
-        onShowMoreRooms: PropTypes.func,
         searchFilter: PropTypes.string,
         emptyContent: PropTypes.node, // content shown if the list is empty
         headerItems: PropTypes.node, // content shown in the sublist header
@@ -71,8 +70,6 @@ const RoomSubList = React.createClass({
         return {
             onHeaderClick: function() {
             }, // NOP
-            onShowMoreRooms: function() {
-            }, // NOP
             extraTiles: [],
             isInvite: false,
             showEmpty: true,
@@ -138,7 +135,6 @@ const RoomSubList = React.createClass({
             // The header isCollapsable, so the click is to be interpreted as collapse and truncation logic
             const isHidden = !this.state.hidden;
             this.setState({hidden: isHidden});
-            this.props.onShowMoreRooms();
             this.props.onHeaderClick(isHidden);
         } else {
             // The header is stuck, so the click is to be interpreted as a scroll to the header
diff --git a/src/components/views/rooms/RoomList.js b/src/components/views/rooms/RoomList.js
index ef82075e89..005c19053e 100644
--- a/src/components/views/rooms/RoomList.js
+++ b/src/components/views/rooms/RoomList.js
@@ -406,13 +406,6 @@ module.exports = React.createClass({
         }
     },
 
-    onShowMoreRooms: function() {
-        // kick gemini in the balls to get it to wake up
-        // XXX: uuuuuuugh.
-        if (!this._gemScroll) return;
-        this._gemScroll.forceUpdate();
-    },
-
     _getEmptyContent: function(section) {
         if (this.state.selectedTags.length > 0) {
             return null;
@@ -508,9 +501,6 @@ module.exports = React.createClass({
     render: function() {
         const RoomSubList = sdk.getComponent('structures.RoomSubList');
 
-        // XXX: we can't detect device-level (localStorage) settings onChange as the SettingsStore does not notify
-        // so checking on every render is the sanest thing at this time.
-        const showEmpty = SettingsStore.getValue('RoomSubList.showEmpty');
 
         const self = this;
 
@@ -518,7 +508,6 @@ module.exports = React.createClass({
             const defaultProps = {
                 collapsed: self.props.collapsed,
                 searchFilter: self.props.searchFilter,
-                onShowMoreRooms: self.onShowMoreRooms,
                 showEmpty: showEmpty,
                 incomingCall: self.state.incomingCall,
             };