mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Changed breadcrumb event handling to not use delegate 

Using delegate might break apps that embed themselves in the files
container. When an app embeds itself and the user clicks a breadcrumb,
it will simply reload the whole browser page.
This commit is contained in:
Vincent Petry 2013-09-01 14:24:01 +02:00
parent 364e7991a4
commit 3cf0820d35
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
apps/files/js/files.js
View File

@ -131,10 +131,12 @@ Files={
Files.breadcrumbsWidth += $(breadcrumb).get(0).offsetWidth;
});
$.each($('#controls .actions>div'), function(index, action) {
Files.breadcrumbsWidth += $(action).get(0).offsetWidth;
});
// event handlers for breadcrumb items
$('#controls .crumb a').on('click', onClickBreadcrumb);
},
resizeBreadcrumbs: function (width, firstRun) {
@ -349,9 +351,6 @@ $(document).ready(function() {
var width = $(this).width();
Files.resizeBreadcrumbs(width, true);
// event handlers for breadcrumb items
$('#controls').delegate('.crumb a', 'click', onClickBreadcrumb);
// display storage warnings
setTimeout ( "Files.displayStorageWarnings()", 100 );
OC.Notification.setDefault(Files.displayStorageWarnings);