From 0a37f1c4787b2e3a3643dcaeecf946b144d79d34 Mon Sep 17 00:00:00 2001
From: Matthew Hodgson <matthew@matrix.org>
Date: Mon, 15 Feb 2016 09:52:57 +0200
Subject: [PATCH] improve the fix for
 https://github.com/vector-im/vector-web/issues/917

---
 .../views/rooms/SearchableEntityList.js       | 51 ++++++++++---------
 1 file changed, 27 insertions(+), 24 deletions(-)

diff --git a/src/components/views/rooms/SearchableEntityList.js b/src/components/views/rooms/SearchableEntityList.js
index c91e058e4c..fc0446ab41 100644
--- a/src/components/views/rooms/SearchableEntityList.js
+++ b/src/components/views/rooms/SearchableEntityList.js
@@ -140,34 +140,37 @@ var SearchableEntityList = React.createClass({
         }
 
         var list;
-        if (this.props.truncateAt) { // caller wants list truncated
-            var TruncatedList = sdk.getComponent("elements.TruncatedList");
-            list = (
-                <TruncatedList className="mx_SearchableEntityList_list"
-                        truncateAt={this.state.truncateAt} // use state truncation as it may be expanded
-                        createOverflowElement={this._createOverflowEntity}>
-                    {this.state.results.map((entity) => {
-                        return entity.getJsx();
-                    })}
-                </TruncatedList>
-            );
-        }
-        else {
-            list = (
-                <div className="mx_SearchableEntityList_list">
-                    {this.state.results.map((entity) => {
-                        return entity.getJsx();
-                    })}
-                </div>
-            );
+        if (this.state.results.length) {
+            if (this.props.truncateAt) { // caller wants list truncated
+                var TruncatedList = sdk.getComponent("elements.TruncatedList");
+                list = (
+                    <TruncatedList className="mx_SearchableEntityList_list"
+                            truncateAt={this.state.truncateAt} // use state truncation as it may be expanded
+                            createOverflowElement={this._createOverflowEntity}>
+                        {this.state.results.map((entity) => {
+                            return entity.getJsx();
+                        })}
+                    </TruncatedList>
+                );
+            }
+            else {
+                list = (
+                    <div className="mx_SearchableEntityList_list">
+                        {this.state.results.map((entity) => {
+                            return entity.getJsx();
+                        })}
+                    </div>
+                );
+            }
+            list = <GeminiScrollbar autoshow={true} className="mx_SearchableEntityList_listWrapper">
+                        { list }
+                    </GeminiScrollbar>;
         }
 
         return (
             <div className={ "mx_SearchableEntityList " + (this.state.query.length ? "mx_SearchableEntityList_expanded" : "") }>
-                {inputBox}
-                <GeminiScrollbar autoshow={true} className="mx_SearchableEntityList_listWrapper">
-                    { list }
-                </GeminiScrollbar>
+                { inputBox }
+                { list }
                 { this.state.query.length ? <div className="mx_SearchableEntityList_hrWrapper"><hr/></div> : '' }
             </div>
         );