From 9866efabe537ed8053845d40c97a379a8bd58cf7 Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@matrix.org>
Date: Wed, 11 Sep 2019 16:18:34 +0200
Subject: [PATCH] pr feedback

---
 src/components/structures/RoomDirectory.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/components/structures/RoomDirectory.js b/src/components/structures/RoomDirectory.js
index 364b910b46..c62882d097 100644
--- a/src/components/structures/RoomDirectory.js
+++ b/src/components/structures/RoomDirectory.js
@@ -471,7 +471,10 @@ module.exports = React.createClass({
             topic = `${topic.substring(0, MAX_TOPIC_LENGTH)}...`;
         }
         topic = linkifyAndSanitizeHtml(topic);
-
+        const avatarUrl = ContentRepo.getHttpUriForMxc(
+                                MatrixClientPeg.get().getHomeserverUrl(),
+                                room.avatar_url, 24, 24, "crop",
+                            );
         return (
             <tr key={ room.room_id }
                 onClick={() => this.onRoomClicked(room)}
@@ -481,15 +484,13 @@ module.exports = React.createClass({
                 <td className="mx_RoomDirectory_roomAvatar">
                     <BaseAvatar width={24} height={24} resizeMethod='crop'
                         name={ name } idName={ name }
-                        url={ ContentRepo.getHttpUriForMxc(
-                                MatrixClientPeg.get().getHomeserverUrl(),
-                                room.avatar_url, 24, 24, "crop") } />
+                        url={ avatarUrl } />
                 </td>
                 <td className="mx_RoomDirectory_roomDescription">
                     <div className="mx_RoomDirectory_name">{ name }</div>&nbsp;
                     <div className="mx_RoomDirectory_topic"
-                         onClick={ (ev) => { ev.stopPropagation(); } }
-                         dangerouslySetInnerHTML={{ __html: topic }} />
+                        onClick={ (ev) => { ev.stopPropagation(); } }
+                        dangerouslySetInnerHTML={{ __html: topic }} />
                     <div className="mx_RoomDirectory_alias">{ get_display_alias_for_room(room) }</div>
                 </td>
                 <td className="mx_RoomDirectory_roomMemberCount">