From f7c4cca67535810ee0c561a6dc3d1e4e9b0f1ebb Mon Sep 17 00:00:00 2001
From: Kegan Dougal <kegan@matrix.org>
Date: Fri, 18 Dec 2015 13:32:22 +0000
Subject: [PATCH] Add TODO after spending 15 mins trying to figure out the
 difference between props.collapsed and state.hidden

---
 src/components/structures/RoomSubList.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/components/structures/RoomSubList.js b/src/components/structures/RoomSubList.js
index f6f10ec6..b71d3285 100644
--- a/src/components/structures/RoomSubList.js
+++ b/src/components/structures/RoomSubList.js
@@ -64,6 +64,11 @@ var RoomSubList = React.createClass({
         bottommost: React.PropTypes.bool,
         selectedRoom: React.PropTypes.string.isRequired,
         activityMap: React.PropTypes.object.isRequired,
+
+        // TODO: Fix the name of this. This is too easily confused with the
+        // "hidden" state which is the expanded (or not) view of the list of rooms.
+        // What this prop *really* does is control whether the room name is displayed
+        // so it should be named as such.
         collapsed: React.PropTypes.bool.isRequired,
         onHeaderClick: React.PropTypes.func,
         alwaysShowHeader: React.PropTypes.bool