Fixed favorite folder view display

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

2 files changed, 14 insertions, 7 deletions

diff --git a/apps/files/js/app.js b/apps/files/js/app.js
index ec223e418b7..d407c49ea51 100644
--- a/apps/files/js/app.js
+++ b/apps/files/js/app.js
@@ -239,8 +239,8 @@
 			var params;
 			if (e && e.itemId) {
 				params = {
-					view: e.itemId,
-					dir: '/'
+					view: e.view !== '' ? e.view : e.itemId,
+					dir: e.dir ? e.dir : '/'
 				};
 				this._changeUrl(params.view, params.dir);
 				OC.Apps.hideAppSidebar($('.detailsView'));
diff --git a/apps/files/js/navigation.js b/apps/files/js/navigation.js
index 207f9b1c206..d783fcc2156 100644
--- a/apps/files/js/navigation.js
+++ b/apps/files/js/navigation.js
@@ -111,33 +111,40 @@
 		 * @param array options "silent" to not trigger event
 		 */
 		setActiveItem: function (itemId, options) {
+			var currentItem = this.$el.find('li[data-id=' + itemId + ']');
+			var itemDir = currentItem.data('dir');
+			var itemView = currentItem.data('view');
 			var oldItemId = this._activeItem;
 			if (itemId === this._activeItem) {
 				if (!options || !options.silent) {
 					this.$el.trigger(
 						new $.Event('itemChanged', {
 							itemId: itemId,
-							previousItemId: oldItemId
+							previousItemId: oldItemId,
+							dir: itemDir,
+							view: itemView
 						})
 					);
 				}
 				return;
 			}
-			this.$el.find('li').removeClass('active');
+			this.$el.find('li a').removeClass('active');
 			if (this.$currentContent) {
 				this.$currentContent.addClass('hidden');
 				this.$currentContent.trigger(jQuery.Event('hide'));
 			}
 			this._activeItem = itemId;
-			this.$el.find('li[data-id=' + itemId + ']').addClass('active');
-			this.$currentContent = $('#app-content-' + itemId);
+			currentItem.children('a').addClass('active');
+			this.$currentContent = $('#app-content-' + (itemView !== '' ? itemView : itemId));
 			this.$currentContent.removeClass('hidden');
 			if (!options || !options.silent) {
 				this.$currentContent.trigger(jQuery.Event('show'));
 				this.$el.trigger(
 					new $.Event('itemChanged', {
 						itemId: itemId,
-						previousItemId: oldItemId
+						previousItemId: oldItemId,
+						dir: itemDir,
+						view: itemView
 					})
 				);
 			}