diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js index 19578bee8..8b7e21765 100644 --- a/app/assets/javascripts/application.js +++ b/app/assets/javascripts/application.js @@ -586,19 +586,23 @@ function expandScmEntry(id) { function scmEntryClick(id, url) { var el = $('#'+id); + var expander = el.find('.expander'); + var folderIcon = el.find('.icon-folder, .icon-folder-open')[0]; if (el.hasClass('open')) { collapseScmEntry(id); - el.find('.expander').switchClass('icon-expanded', 'icon-collapsed'); + expander.switchClass('icon-expanded', 'icon-collapsed'); el.addClass('collapsed'); - updateSVGIcon(el.find('.icon-folder')[0], 'folder') + updateSVGIcon(expander[0], 'angle-right'); + updateSVGIcon(folderIcon, 'folder'); return false; } else if (el.hasClass('loaded')) { expandScmEntry(id); - el.find('.expander').switchClass('icon-collapsed', 'icon-expanded'); + expander.switchClass('icon-collapsed', 'icon-expanded'); el.removeClass('collapsed'); - updateSVGIcon(el.find('.icon-folder-open')[0], 'folder-open') + updateSVGIcon(expander[0], 'angle-down'); + updateSVGIcon(folderIcon, 'folder-open'); return false; } @@ -611,8 +615,9 @@ function scmEntryClick(id, url) { success: function(data) { el.after(data); el.addClass('open').addClass('loaded').removeClass('loading'); - updateSVGIcon(el.find('.icon-folder')[0], 'folder-open') - el.find('.expander').switchClass('icon-collapsed', 'icon-expanded'); + updateSVGIcon(expander[0], 'angle-down'); + updateSVGIcon(folderIcon, 'folder-open'); + expander.switchClass('icon-collapsed', 'icon-expanded'); } }); return true;