Skip to content
Snippets Groups Projects

Issue #3449491: ResponsiveMenus sometimes adds multiple click handlers to the toggle and accordion controls breaking them

Open Issue #3449491: ResponsiveMenus sometimes adds multiple click handlers to the toggle and accordion controls breaking them
Open James Monroe requested to merge issue/adaptivetheme-3449491:7.0.x into 7.0.x
2 files
+ 9
8
Compare changes
  • Side-by-side
  • Inline
Files
2
@@ -2,7 +2,7 @@
@@ -2,7 +2,7 @@
* @file
* @file
* Responsive menus.
* Responsive menus.
*/
*/
(function ($, document, window) {
(function ($, document, window, once) {
"use strict";
"use strict";
@@ -50,7 +50,7 @@
@@ -50,7 +50,7 @@
}
}
});
});
}
}
$(tl, context).on('click', toggleClick);
$(once('atrm-toggle', tl, context)).on('click', toggleClick);
// Accordion toggles.
// Accordion toggles.
function accordionClick(e) {
function accordionClick(e) {
@@ -94,7 +94,7 @@
@@ -94,7 +94,7 @@
if (acd_def == true && acd_load == true) {
if (acd_def == true && acd_load == true) {
$('.rm-block .menu-level-1').addClass('ms-accordion');
$('.rm-block .menu-level-1').addClass('ms-accordion');
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$('.ms-accordion .rm-accordion-trigger', context).on('click', accordionClick);
$(once('atrm-accordion', '.ms-accordion .rm-accordion-trigger', context)).on('click', accordionClick);
}
}
},
},
// The resp menu system only uses one breakpoint, if it matches this
// The resp menu system only uses one breakpoint, if it matches this
@@ -109,7 +109,7 @@
@@ -109,7 +109,7 @@
if (acd_both == false) {
if (acd_both == false) {
$('.rm-block .menu-level-1').addClass('ms-accordion');
$('.rm-block .menu-level-1').addClass('ms-accordion');
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$('.ms-accordion .rm-accordion-trigger', context).on('click', accordionClick);
$(once('atrm-accordion', '.ms-accordion .rm-accordion-trigger', context)).on('click', accordionClick);
}
}
} else {
} else {
$('.ms-accordion .rm-accordion-trigger').remove();
$('.ms-accordion .rm-accordion-trigger').remove();
@@ -135,7 +135,7 @@
@@ -135,7 +135,7 @@
if (acd_both == false) {
if (acd_both == false) {
$('.rm-block .menu-level-1').addClass('ms-accordion');
$('.rm-block .menu-level-1').addClass('ms-accordion');
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$.ready(copyButtons('.ms-accordion .is-parent__wrapper .menu__link'));
$('.ms-accordion .rm-accordion-trigger', context).on('click', accordionClick);
$(once('atrm-accordion', '.ms-accordion .rm-accordion-trigger', context)).on('click', accordionClick);
}
}
} else {
} else {
$('.ms-accordion .rm-accordion-trigger').remove();
$('.ms-accordion .rm-accordion-trigger').remove();
@@ -157,4 +157,4 @@
@@ -157,4 +157,4 @@
});
});
}
}
};
};
}(jQuery, document, window));
}(jQuery, document, window, once));
Loading