diff --git a/apps/files/js/detailsview.js b/apps/files/js/detailsview.js index ac10b8380e..aed1736693 100644 --- a/apps/files/js/detailsview.js +++ b/apps/files/js/detailsview.js @@ -118,10 +118,13 @@ */ render: function() { // remove old instances - if ($('#app-sidebar').length === 0) { + var $appSidebar = $('#app-sidebar'); + if ($appSidebar.length === 0) { this.$el.insertAfter($('#app-content')); } else { - $('#app-sidebar').replaceWith(this.$el) + if ($appSidebar[0] !== this.el) { + $appSidebar.replaceWith(this.$el) + } } var templateVars = { diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index f44a36435a..8e7c60551a 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -564,7 +564,7 @@ } this._currentFileModel = model; - + this._detailsView.render(); this._detailsView.setFileInfo(model); this._detailsView.$el.scrollTop(0); }, @@ -1623,7 +1623,7 @@ // discard finished uploads list, we'll get it through a regular reload this._uploads = {}; - this.reload().then(function(success){ + return this.reload().then(function(success){ if (!success) { self.changeDirectory(currentDir, true); } diff --git a/tests/acceptance/features/app-files.feature b/tests/acceptance/features/app-files.feature index 956430c02b..00f09900d3 100644 --- a/tests/acceptance/features/app-files.feature +++ b/tests/acceptance/features/app-files.feature @@ -1,15 +1,23 @@ Feature: app-files - Scenario: viewing a favorite file in its folder closes the details view + Scenario: viewing a favorite file in its folder shows the correct sidebar view Given I am logged in + And I create a new folder named "other" + And I mark "other" as favorite And I mark "welcome.txt" as favorite + And I see that "other" is marked as favorite And I see that "welcome.txt" is marked as favorite And I open the "Favorites" section - And I open the details view for "welcome.txt" + And I open the details view for "other" And I see that the details view is open + And I see that the file name shown in the details view is "other" When I view "welcome.txt" in folder Then I see that the current section is "All files" - And I see that the details view is closed + And I see that the details view is open + And I see that the file name shown in the details view is "welcome.txt" + When I open the details view for "other" + And I see that the file name shown in the details view is "other" + Scenario: viewing a favorite file in its folder does not prevent opening the details view in "All files" section Given I am logged in