@salesforcedevs/dx-components 1.3.224 → 1.3.227

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforcedevs/dx-components",
-    "version": "1.3.224",
+    "version": "1.3.227",
     "description": "DX Lightning web components",
     "license": "MIT",
     "engines": {
@@ -44,5 +44,5 @@
     "volta": {
         "node": "16.19.1"
     },
-    "gitHead": "d5ff886eb27dbd4b076db913c1c4efbdf188bf57"
+    "gitHead": "ca4d2f11924db69bc1e872fb61587af9bb78cdde"
 }

package/src/modules/dx/searchResults/searchResults.ts

@@ -8,6 +8,9 @@ import {
 import { getContentTypeColorVariables } from "dxUtils/contentTypes";
 import { pollUntil } from "dxUtils/async";
 
+// Max height for breadcrumb without wrapping
+const MAX_BREADCRUMB_HEIGHT = 16;
+
 interface CoveoSearch {
     state: typeof CoveoSDK.state;
     get: typeof CoveoSDK.get;
@@ -141,10 +144,10 @@ const processParts = (parts: string[], internalFlag = false) => {
     return parts.filter(filterFn).map((part) => {
         // Remove special characters & .htm/.xml extension
         part = part
-            .replace(/_/g, " ")
-            .replace(/-/g, " ")
-            .replace(/.html*/g, " ")
-            .replace(/.xml/g, " ")
+            .replace(/_/g, "")
+            .replace(/-/g, "")
+            .replace(/.html*/g, "")
+            .replace(/.xml/g, "")
             .replace(/b2c/g, "B2C");
 
         // Capitalize first letter of each word
@@ -367,17 +370,16 @@ export default class SearchResults extends LightningElement {
                 1000
             );
 
-            this.processBreadcrumbs(this.root!);
+            this.processBreadcrumbs(root);
 
             window.onresize = () => this.processBreadcrumbs(root);
         });
     }
 
-    // Checks if text is wrapping by comparing it with an element's text that doesn't wrap
-    private isTextWrapping = (
-        elementOne: HTMLElement,
-        elementTwo: HTMLElement
-    ) => elementOne.offsetHeight > elementTwo.offsetHeight;
+    // check if the breadcrumb is overflowing or not based on the height of a single character in the text element
+    private isTextWrapping = (element: HTMLElement) => {
+        return element.offsetHeight > MAX_BREADCRUMB_HEIGHT;
+    };
 
     private truncateBreadcrumbText = (breadcrumbItems: HTMLElement[]) => {
         breadcrumbItems.forEach((breadcrumbItem: HTMLElement) => {
@@ -396,12 +398,29 @@ export default class SearchResults extends LightningElement {
         breadcrumb: HTMLElement
     ) => {
         for (let i = 1; i < breadcrumbItems.length; i++) {
-            if (this.isTextWrapping(breadcrumb, breadcrumbItems[0])) {
-                breadcrumbItems[i].textContent = "...";
+            if (this.isTextWrapping(breadcrumb)) {
+                // if the previous element is an ellipsis, make it empty (in order to avoid multiple grouped ellipsis)
+                if (breadcrumbItems[i - 1]?.textContent === "...") {
+                    breadcrumbItems[i].innerHTML = "";
+                } else {
+                    breadcrumbItems[i].textContent = "...";
+                }
             } else {
                 break; // Exit the loop if the breadcrumb is no longer overflowing
             }
         }
+
+        // remove any empty breadcrumb items
+        breadcrumb.innerHTML = breadcrumb.innerHTML
+            .replace(
+                / ?\/ +<span class="breadcrumb-item"><\/span> +\/ ?/g,
+                " / "
+            )
+            // when first loading the page on mobile, the breadcrumb items are not grouped correctly
+            .replace(
+                `<span class="breadcrumb-item">...</span> / <span class="breadcrumb-item">...</span>`,
+                `<span class="breadcrumb-item">...</span>`
+            );
     };
 
     private formatBreadcrumbs = (breadcrumbs: HTMLElement[]) => {
@@ -411,7 +430,7 @@ export default class SearchResults extends LightningElement {
                 breadcrumb.querySelectorAll(".breadcrumb-item");
 
             // Check if the breadcrumb is overflowing by comparing it's height to the height of the first breadcrumb item
-            if (this.isTextWrapping(breadcrumb, breadcrumbItems[0])) {
+            if (this.isTextWrapping(breadcrumb)) {
                 // it is overflowing, so we need to truncate long titles to 30 characters
                 this.truncateBreadcrumbText(breadcrumbItems);
 
@@ -419,7 +438,7 @@ export default class SearchResults extends LightningElement {
                 this.addBreadcrumbEllipsis(breadcrumbItems, breadcrumb);
 
                 // After processing all breadcrumb items, if it's still overflowing, hide the breadcrumb element
-                if (this.isTextWrapping(breadcrumb, breadcrumbItems[0])) {
+                if (this.isTextWrapping(breadcrumb)) {
                     breadcrumb.style.display = "none";
                 }
             }