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.

diff --git a/apps/files/js/files.js b/apps/files/js/files.js
index 1ea2f5fbcc276681aba39fee433ca4f7eda48855..7aef8ea1d114a44a8520a6221b3cfca21e9fe867 100644 (file)
--- a/apps/files/js/files.js
+++ b/apps/files/js/files.js
@@ -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);