From 8c968d9f4f8a339705fa4dab64368d5c955e02f3 Mon Sep 17 00:00:00 2001
From: Jori Lallo <jtlallo@gmail.com>
Date: Thu, 16 Feb 2017 23:04:52 -0800
Subject: [PATCH] Correct naming of variables

---
 .../scenes/DocumentScene/DocumentSceneStore.js   | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/frontend/scenes/DocumentScene/DocumentSceneStore.js b/frontend/scenes/DocumentScene/DocumentSceneStore.js
index f0be2da1b..2b3c27890 100644
--- a/frontend/scenes/DocumentScene/DocumentSceneStore.js
+++ b/frontend/scenes/DocumentScene/DocumentSceneStore.js
@@ -9,7 +9,7 @@ class DocumentSceneStore {
   static cache;
 
   @observable document;
-  @observable openNodes = [];
+  @observable collapsedNodes = [];
 
   @observable isFetching = true;
   @observable updatingContent = false;
@@ -28,9 +28,7 @@ class DocumentSceneStore {
     const tree = this.document.collection.navigationTree;
 
     const collapseNodes = (node) => {
-      if (this.openNodes.includes(node.id)) {
-        node.collapsed = true;
-      }
+      node.collapsed = this.collapsedNodes.includes(node.id);
       node.children = node.children.map(childNode => {
         return collapseNodes(childNode);
       });
@@ -109,10 +107,10 @@ class DocumentSceneStore {
   }
 
   @action onNodeCollapse = (nodeId) => {
-    if (_.indexOf(this.openNodes, nodeId) >= 0) {
-      this.openNodes = _.without(this.openNodes, nodeId);
+    if (_.indexOf(this.collapsedNodes, nodeId) >= 0) {
+      this.collapsedNodes = _.without(this.collapsedNodes, nodeId);
     } else {
-      this.openNodes.push(nodeId);
+      this.collapsedNodes.push(nodeId);
     }
   }
 
@@ -120,13 +118,13 @@ class DocumentSceneStore {
 
   persistSettings = () => {
     localStorage[DOCUMENT_PREFERENCES] = JSON.stringify({
-      openNodes: toJS(this.openNodes),
+      collapsedNodes: toJS(this.collapsedNodes),
     });
   }
 
   constructor(settings, options) {
     // Rehydrate settings
-    this.openNodes = settings.openNodes || [];
+    this.collapsedNodes = settings.collapsedNodes || [];
     this.cache = options.cache;
 
     // Persist settings to localStorage