From 7f6941677d32523d4ad1d93a1044945c097f21e0 Mon Sep 17 00:00:00 2001 From: Paulo Reis Date: Tue, 21 Feb 2017 14:43:05 +0000 Subject: [PATCH] Check local storage on load; use explicit condition syntax. --- .../review-panel/controllers/ReviewPanelController.coffee | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee index d740eb02e3..5067799571 100644 --- a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee +++ b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee @@ -19,7 +19,7 @@ define [ openSubView: $scope.SubViews.CUR_FILE overview: loading: false - docsCollapsedState: {} + docsCollapsedState: JSON.parse(localStorage("docs_collapsed_state:#{$scope.project_id}")) or {} dropdown: loading: false commentThreads: {} @@ -28,13 +28,11 @@ define [ rendererData: {} loadingThreads: false - $scope.$on "project:joined", () -> - $scope.reviewPanel.overview.docsCollapsedState = JSON.parse(localStorage("docs_collapsed_state:#{$scope.project_id}")) or {} - window.addEventListener "beforeunload", () -> collapsedStates = {} for doc, state of $scope.reviewPanel.overview.docsCollapsedState - collapsedStates[doc] = state if state is true + if state + collapsedStates[doc] = state valToStore = if Object.keys(collapsedStates).length > 0 then JSON.stringify(collapsedStates) else null localStorage("docs_collapsed_state:#{$scope.project_id}", valToStore)