From 760493b15a3b4dfdab29b9558aabca6453451d32 Mon Sep 17 00:00:00 2001
From: Julia Radzhabova <Julia.Radzhabova@onlyoffice.com>
Date: Wed, 20 Dec 2017 14:09:31 +0300
Subject: [PATCH] Fix expand/collapse for treeview.

---
 apps/common/main/lib/component/TreeView.js    | 29 ++++++++++++++-----
 apps/common/main/resources/less/treeview.less | 16 +++++++++-
 2 files changed, 36 insertions(+), 9 deletions(-)

diff --git a/apps/common/main/lib/component/TreeView.js b/apps/common/main/lib/component/TreeView.js
index 4b4ed903f..fd89c1f9d 100644
--- a/apps/common/main/lib/component/TreeView.js
+++ b/apps/common/main/lib/component/TreeView.js
@@ -65,7 +65,7 @@ define([
     Common.UI.TreeViewStore = Backbone.Collection.extend({
         model: Common.UI.TreeViewModel,
 
-        expand: function(record) {
+        expandSubItems: function(record) {
             var me = this;
             var _expand_sub_items = function(idx, expanded, level) {
                 for (var i=idx+1; i<me.length; i++) {
@@ -86,7 +86,7 @@ define([
             _expand_sub_items(record.get('index'), true, record.get('level'));
         },
 
-        collapse: function(record) {
+        collapseSubItems: function(record) {
             var start_level = record.get('level'),
                 index = record.get('index');
             for (var i=index+1; i<this.length; i++) {
@@ -98,6 +98,22 @@ define([
                     break;
                 }
             }
+            return i-1;
+        },
+
+        expandAll: function() {
+            this.each(function(item) {
+                item.set('isExpanded', true);
+                item.set('isVisible', true);
+            });
+        },
+
+        collapseAll: function() {
+            for (var i=0; i<this.length; i++) {
+                var item = this.at(i);
+                item.set('isExpanded', false);
+                i = this.collapseSubItems(item);
+            }
         }
     });
 
@@ -176,22 +192,19 @@ define([
 
                     var isExpanded = !record.get('isExpanded');
                     record.set('isExpanded', isExpanded);
-                    this.store[(isExpanded) ? 'expand' : 'collapse'](record);
+                    this.store[(isExpanded) ? 'expandSubItems' : 'collapseSubItems'](record);
                     this.scroller.update({minScrollbarLength: 40});
                 } else
                     Common.UI.DataView.prototype.onClickItem.call(this, view, record, e);
             },
 
             expandAll: function() {
-                this.store.each(function(item) {
-                    item.set('isExpanded', true);
-                });
-                this.store.expand(this.store.at(0));
+                this.store.expandAll();
                 this.scroller.update({minScrollbarLength: 40});
             },
 
             collapseAll: function() {
-                this.store.collapse(this.store.at(0));
+                this.store.collapseAll();
                 this.scroller.update({minScrollbarLength: 40});
             }
         }
diff --git a/apps/common/main/resources/less/treeview.less b/apps/common/main/resources/less/treeview.less
index 7ecb01b1d..e47c2303e 100644
--- a/apps/common/main/resources/less/treeview.less
+++ b/apps/common/main/resources/less/treeview.less
@@ -1,5 +1,19 @@
 .treeview {
-  .item {
+  &.inner {
+    width: 100%;
+    height: 100%;
+    position: relative;
+    overflow: hidden;
+
+    .empty-text {
+      text-align: center;
+      height: 100%;
+      width: 100%;
+      color: #b2b2b2;
+    }
+  }
+
+  > .item {
     display: block;
     width: 100%;
     .box-shadow(none);