From bec6ea20f6933f07cb616c29ce72efc076a97dc0 Mon Sep 17 00:00:00 2001 From: Phil Hughes Date: Thu, 17 Aug 2017 14:00:50 +0100 Subject: fixed breadcrumbs being undefined --- app/assets/javascripts/breadcrumb.js | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) (limited to 'app/assets/javascripts/breadcrumb.js') diff --git a/app/assets/javascripts/breadcrumb.js b/app/assets/javascripts/breadcrumb.js index 97340953c77..7433fcbbad1 100644 --- a/app/assets/javascripts/breadcrumb.js +++ b/app/assets/javascripts/breadcrumb.js @@ -8,14 +8,17 @@ export const addTooltipToEl = (el) => { export default () => { const breadcrumbs = document.querySelector('.breadcrumbs-list'); - const topLevelLinks = breadcrumbs.querySelectorAll('.breadcrumbs-list > li > a'); - const $expander = $('.js-breadcrumbs-collapsed-expander'); - topLevelLinks.forEach(el => addTooltipToEl(el)); + if (breadcrumbs) { + const topLevelLinks = breadcrumbs.querySelectorAll('.breadcrumbs-list > li > a'); + const $expander = $('.js-breadcrumbs-collapsed-expander'); - $expander.closest('.dropdown') - .on('show.bs.dropdown hide.bs.dropdown', (e) => { - $('.js-breadcrumbs-collapsed-expander', e.currentTarget).toggleClass('open') - .tooltip('hide'); - }); + topLevelLinks.forEach(el => addTooltipToEl(el)); + + $expander.closest('.dropdown') + .on('show.bs.dropdown hide.bs.dropdown', (e) => { + $('.js-breadcrumbs-collapsed-expander', e.currentTarget).toggleClass('open') + .tooltip('hide'); + }); + } }; -- cgit v1.2.1