Skip to content

Commit

Permalink
fix(cxl-ui): (cxl-marketing-nav) handle errors while adding classes t…
Browse files Browse the repository at this point in the history
…o menu items
  • Loading branch information
freudFlintstone committed Nov 30, 2023
1 parent f6e87ef commit b2415bb
Showing 1 changed file with 15 additions and 3 deletions.
18 changes: 15 additions & 3 deletions packages/cxl-ui/src/components/cxl-marketing-nav.js
Original file line number Diff line number Diff line change
Expand Up @@ -265,9 +265,21 @@ export class CXLMarketingNavElement extends LitElement {

// If there are classes, add them, avoiding empty strings.
if (classes) {
classes.forEach((className) => {
if (className) menuItemElement.classList.add(className);
});
try {
classes.forEach((className) => {
if (className) menuItemElement.classList.add(className);
});
} catch (error) {
classes.forEach((className) => {
if (className.indexOf(' ') !== -1) {
// eslint-disable-next-line no-console
console.warn(`One of the classes items contains a space: ${className}`);
className.split(' ').forEach((splitClass) => {
if (splitClass) menuItemElement.classList.add(splitClass);
});
}
});
}

if (classes?.includes('menu-item-search')) {
menuItemElement.addEventListener('click', this.toggleSearchDialog.bind(this));
Expand Down

0 comments on commit b2415bb

Please sign in to comment.