@coveo/quantic 3.44.0 → 3.44.2

package/force-app/main/default/classes/OrganizationEndpointsTest.cls

@@ -5,17 +5,36 @@ public class OrganizationEndpointsTest {
     String orgId = 'testOrgId';
     String env = 'prod';
 
-    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(orgId, env);
+    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(
+      orgId,
+      env
+    );
 
     String expectedPlatform = 'https://testOrgId.org.coveo.com';
     String expectedAnalytics = 'https://testOrgIdanalytics.org.coveo.com';
     String expectedSearch = expectedPlatform + '/rest/search/v2';
     String expectedAdmin = 'https://testOrgIdadmin.org.coveo.com';
 
-    System.assertEquals(expectedPlatform, endpoints.get('platform'), 'Platform URL mismatch for prod');
-    System.assertEquals(expectedAnalytics, endpoints.get('analytics'), 'Analytics URL mismatch for prod');
-    System.assertEquals(expectedSearch, endpoints.get('search'), 'Search URL mismatch for prod');
-    System.assertEquals(expectedAdmin, endpoints.get('admin'), 'Admin URL mismatch for prod');
+    System.assertEquals(
+      expectedPlatform,
+      endpoints.get('platform'),
+      'Platform URL mismatch for prod'
+    );
+    System.assertEquals(
+      expectedAnalytics,
+      endpoints.get('analytics'),
+      'Analytics URL mismatch for prod'
+    );
+    System.assertEquals(
+      expectedSearch,
+      endpoints.get('search'),
+      'Search URL mismatch for prod'
+    );
+    System.assertEquals(
+      expectedAdmin,
+      endpoints.get('admin'),
+      'Admin URL mismatch for prod'
+    );
   }
 
   @isTest
@@ -23,33 +42,70 @@ public class OrganizationEndpointsTest {
     String orgId = 'testOrgId';
     String env = 'dev';
 
-    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(orgId, env);
+    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(
+      orgId,
+      env
+    );
 
     String expectedPlatform = 'https://testOrgId.orgdev.coveo.com';
     String expectedAnalytics = 'https://testOrgIdanalytics.orgdev.coveo.com';
     String expectedSearch = expectedPlatform + '/rest/search/v2';
     String expectedAdmin = 'https://testOrgIdadmin.orgdev.coveo.com';
 
-    System.assertEquals(expectedPlatform, endpoints.get('platform'), 'Platform URL mismatch for dev');
-    System.assertEquals(expectedAnalytics, endpoints.get('analytics'), 'Analytics URL mismatch for dev');
-    System.assertEquals(expectedSearch, endpoints.get('search'), 'Search URL mismatch for dev');
-    System.assertEquals(expectedAdmin, endpoints.get('admin'), 'Admin URL mismatch for dev');
+    System.assertEquals(
+      expectedPlatform,
+      endpoints.get('platform'),
+      'Platform URL mismatch for dev'
+    );
+    System.assertEquals(
+      expectedAnalytics,
+      endpoints.get('analytics'),
+      'Analytics URL mismatch for dev'
+    );
+    System.assertEquals(
+      expectedSearch,
+      endpoints.get('search'),
+      'Search URL mismatch for dev'
+    );
+    System.assertEquals(
+      expectedAdmin,
+      endpoints.get('admin'),
+      'Admin URL mismatch for dev'
+    );
   }
 
   @isTest
   static void testGetOrganizationEndpointsWithoutEnv() {
     String orgId = 'testOrgId';
 
-    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(orgId);
+    Map<String, String> endpoints = OrganizationEndpoints.getOrganizationEndpoints(
+      orgId
+    );
 
     String expectedPlatform = 'https://testOrgId.org.coveo.com';
     String expectedAnalytics = 'https://testOrgIdanalytics.org.coveo.com';
     String expectedSearch = expectedPlatform + '/rest/search/v2';
     String expectedAdmin = 'https://testOrgIdadmin.org.coveo.com';
 
-    System.assertEquals(expectedPlatform, endpoints.get('platform'), 'Platform URL mismatch');
-    System.assertEquals(expectedAnalytics, endpoints.get('analytics'), 'Analytics URL mismatch');
-    System.assertEquals(expectedSearch, endpoints.get('search'), 'Search URL mismatch');
-    System.assertEquals(expectedAdmin, endpoints.get('admin'), 'Admin URL mismatch');
+    System.assertEquals(
+      expectedPlatform,
+      endpoints.get('platform'),
+      'Platform URL mismatch'
+    );
+    System.assertEquals(
+      expectedAnalytics,
+      endpoints.get('analytics'),
+      'Analytics URL mismatch'
+    );
+    System.assertEquals(
+      expectedSearch,
+      endpoints.get('search'),
+      'Search URL mismatch'
+    );
+    System.assertEquals(
+      expectedAdmin,
+      endpoints.get('admin'),
+      'Admin URL mismatch'
+    );
   }
 }

package/force-app/main/default/classes/RecommendationsControllerTest.cls

@@ -1,7 +1,7 @@
 @isTest
 private class RecommendationsControllerTest {
   // This API key is intentionally public — it belongs to a sample organization used for samples/docs.
-  static final String sampleHeadlessConfiguration = '{"accessToken":"xx564559b1-0045-48e1-953c-3addd1ee4457","organizationId":"searchuisamples"}';
+  static final String sampleHeadlessConfiguration = '{"accessToken":"xxb9c21134-8dad-4d53-8a5e-855bacddba8f","organizationId":"searchuisamples"}';
   @IsTest
   static void shouldReturnStringifiedConfiguration() {
     Test.startTest();

package/force-app/main/default/lwc/quanticNumberButton/quanticNumberButton.js

@@ -32,7 +32,11 @@ export default class QuanticNumberButton extends LightningElement {
   }
 
   get buttonClasses() {
-    const classes = ['slds-button', 'slds-m-left_xx-small'];
+    const classes = [
+      'slds-button',
+      'slds-m-left_xx-small',
+      'slds-m-bottom_x-small',
+    ];
     classes.push(
       this.selected ? 'slds-button_brand' : 'slds-button_outline-brand'
     );

package/force-app/main/default/lwc/quanticPager/quanticPager.html

@@ -4,14 +4,17 @@
     </c-quantic-component-error>
   </template>
   <template lwc:else>
-    <template if:true={hasResults}>
-      <lightning-button-group>
+    <template lwc:if={hasResults}>
+      <div
+        class="slds-grid slds-wrap slds-grid_vertical-align-center"
+        data-testid="pager-container"
+      >
         <lightning-button
           variant="base"
           disabled={previousDisabled}
           title={labels.previousPage}
           icon-name="utility:left"
-          class="slds-m-left_x-small"
+          class="slds-m-left_x-small slds-m-bottom_x-small"
           onclick={previous}
           aria-label={labels.previousPage}
           data-testid="previous-button"
@@ -32,12 +35,12 @@
           variant="base"
           title={labels.nextPage}
           icon-name="utility:right"
-          class="slds-m-left_x-small"
+          class="slds-m-left_x-small slds-m-bottom_x-small"
           onclick={next}
           aria-label={labels.nextPage}
           data-testid="next-button"
         ></lightning-button>
-      </lightning-button-group>
+      </div>
     </template>
   </template>
 </template>

package/force-app/main/default/lwc/quanticTabBar/quanticTabBar.js

@@ -1,5 +1,4 @@
 import more from '@salesforce/label/c.quantic_More';
-import {getAbsoluteWidth} from 'c/quanticUtils';
 import {LightningElement, api} from 'lwc';
 
 /**
@@ -39,11 +38,25 @@ export default class QuanticTabBar extends LightningElement {
   maxMoreButtonWidth = 0;
   /** @type {boolean} */
   expandedMoreButton = true;
+  /** @type {Array<Element>} */
+  _overflowingTabs = [];
+  /** @type {Array<Element>} */
+  _displayedTabs = [];
+  /** @type {boolean} */
+  _updateDisplayDirty = false;
+  /** @type {boolean} */
+  _updateScheduled = false;
+
+  /**
+   * Rects captured for the current layout pass.
+   * @type {{container: DOMRect, moreButton: DOMRect, tabs: Map<Element, DOMRect>}|null}
+   */
+  _layoutRects = null;
 
   connectedCallback() {
     window.addEventListener('click', this.closeDropdown);
     window.addEventListener('resize', this.updateTabsDisplay);
-    this.addEventListener('quantic__tabrendered', this.updateTabsDisplay);
+    this.addEventListener('quantic__tabrendered', this.handleTabRendered);
   }
 
   renderedCallback() {
@@ -53,25 +66,118 @@ export default class QuanticTabBar extends LightningElement {
     }
   }
 
+  handleTabRendered = () => {
+    this._updateDisplayDirty = true;
+
+    if (this._updateScheduled) {
+      return;
+    }
+    this._updateScheduled = true;
+    // eslint-disable-next-line @lwc/lwc/no-async-operation
+    requestAnimationFrame(() => {
+      this._updateScheduled = false;
+      if (this._updateDisplayDirty) {
+        this._updateDisplayDirty = false;
+        this.updateTabsDisplay();
+      }
+    });
+  };
+
   /**
    * Updates the display of the tabs.
+   * Computes a single layout snapshot up front to avoid redundant `getBoundingClientRect` reads.
    * @returns {void}
    */
   updateTabsDisplay = () => {
-    this.updateMoreButtonVisibility(this.isOverflow);
-    this.updateTabVisibility(this.overflowingTabs, false);
-    this.updateTabVisibility(this.displayedTabs, true);
-    this.updateDropdownOptions();
-    this.updateMoreButtonPosition();
+    const tabElements = this.getTabsFromSlot();
+    this._layoutRects = {
+      container: this.container.getBoundingClientRect(),
+      moreButton: this.moreButton?.getBoundingClientRect(),
+      tabs: new Map(
+        tabElements.map((tab) => [tab, tab.getBoundingClientRect()])
+      ),
+    };
+
+    const containerWidth = this._layoutRects.container.width;
+    const slotContentWidth = this.computeSlotContentWidth(tabElements);
+    const tabsCount = tabElements.length;
+    const isOverflow = slotContentWidth > containerWidth;
+    // Must run before categorizeTabsVisibility: while display:none, moreButtonWidth reads as 0.
+    this.updateMoreButtonVisibility(isOverflow);
+    this._layoutRects.moreButton = this.moreButton?.getBoundingClientRect();
+    this.updateMoreButtonState();
+
+    const {overflowingTabs, displayedTabs} = this.categorizeTabsVisibility(
+      tabElements,
+      isOverflow
+    );
+    this._overflowingTabs = overflowingTabs;
+    this._displayedTabs = displayedTabs;
+
+    this.updateTabVisibility(overflowingTabs, false, tabsCount);
+    this.updateTabVisibility(displayedTabs, true, tabsCount);
+    this.updateDropdownOptions(overflowingTabs);
+    this.updateMoreButtonPosition(displayedTabs);
     this.isDropdownOpen = false;
+    this._layoutRects = null;
   };
 
+  /**
+   * Computes, in a single pass, all the layout values needed to update the tabs display.
+   * @param {Array<Element>} tabElements
+   * @param {boolean} isOverflow Whether the tabs overflow the container.
+   * @returns {{overflowingTabs: Array<Element>, displayedTabs: Array<Element>}}
+   */
+  categorizeTabsVisibility(tabElements, isOverflow) {
+    // @ts-ignore
+    const selectedTab = tabElements.find((el) => el.isActive);
+    const selectedTabWidth = Math.ceil(
+      this._layoutRects.tabs.get(selectedTab)?.width ?? 0
+    );
+    const moreButton = this.moreButton?.querySelector('button');
+    const moreButtonWidth = Math.max(
+      this._layoutRects.moreButton?.width ?? 0,
+      moreButton?.offsetWidth ?? 0,
+      moreButton?.scrollWidth ?? 0
+    );
+    const containerWidth = this._layoutRects.container.width;
+    const displayedTabs = [];
+    const overflowingTabs = [];
+    let displayedTabsWidth = 0;
+    let selectedTabDisplayed = false;
+
+    tabElements.forEach((tab) => {
+      const tabWidth = Math.ceil(this._layoutRects.tabs.get(tab)?.width ?? 0);
+      // @ts-ignore
+      if (tab.isActive) {
+        displayedTabs.push(tab);
+        displayedTabsWidth += tabWidth;
+        selectedTabDisplayed = true;
+        return;
+      }
+
+      const reservedWidth = isOverflow
+        ? moreButtonWidth + (selectedTabDisplayed ? 0 : selectedTabWidth)
+        : 0;
+      if (displayedTabsWidth + tabWidth + reservedWidth <= containerWidth) {
+        displayedTabs.push(tab);
+        displayedTabsWidth += tabWidth;
+      } else {
+        overflowingTabs.push(tab);
+      }
+    });
+
+    return {overflowingTabs, displayedTabs};
+  }
+
   /**
    * Updates the dropdown options.
+   * @param {Array<Element>} [overflowingTabs]
    * @returns {void}
    */
-  updateDropdownOptions() {
-    this.tabsInDropdown = this.overflowingTabs.map((el, index) => ({
+  updateDropdownOptions(overflowingTabs) {
+    // @ts-ignore
+    this.tabsInDropdown = overflowingTabs.map((el, index) => ({
       id: index,
       // @ts-ignore
       label: el.label,
@@ -82,14 +188,19 @@ export default class QuanticTabBar extends LightningElement {
 
   /**
    * Updates the position of the "More" button element.
-   * We need to update the position of the "More" button so that it is always to the right of the last tab displayed, as hidden tabs are just hidden visually but there is always space allocated for them.
+   * The displayed tabs are assigned contiguous flex orders below, so their widths determine the
+   * position of More even before the browser applies the new order.
+   * @param {Array<Element>} [displayedTabs] Optionally pass a pre-computed list of displayed tabs
+   *  to avoid recomputing it from the live DOM.
    * @returns {void}
    */
-  updateMoreButtonPosition() {
-    this.moreButton?.style.setProperty(
-      'left',
-      `${this.displayedTabs.length ? this.lastVisibleTabRightPosition : 0}px`
+  updateMoreButtonPosition(displayedTabs) {
+    const position = displayedTabs.reduce(
+      (total, tab) =>
+        total + Math.ceil(this._layoutRects.tabs.get(tab)?.width ?? 0),
+      0
     );
+    this.moreButton?.style.setProperty('left', `${position}px`);
   }
 
   /**
@@ -98,12 +209,18 @@ export default class QuanticTabBar extends LightningElement {
    */
   updateMoreButtonState() {
     if (this.hasRendered) {
-      if (this.maxMoreButtonWidth < this.moreButtonWidth) {
-        this.maxMoreButtonWidth = this.moreButtonWidth;
+      const moreButtonWidth = this._layoutRects.moreButton?.width ?? 0;
+      const selectedTabWidth = Math.ceil(
+        this._layoutRects.tabs.get(this.selectedTab)?.width ?? 0
+      );
+
+      if (this.maxMoreButtonWidth < moreButtonWidth) {
+        this.maxMoreButtonWidth = moreButtonWidth;
       }
 
       this.expandedMoreButton =
-        this.containerWidth > this.maxMoreButtonWidth + this.selectedTabWidth;
+        this._layoutRects.container.width >
+        this.maxMoreButtonWidth + selectedTabWidth;
     }
   }
 
@@ -120,28 +237,24 @@ export default class QuanticTabBar extends LightningElement {
    * Updates the tabs visibility.
    * @param {Array<Element>} tabElements
    * @param {boolean} isVisible
+   * @param {number} [totalTabsCount] Optionally pass the total tab count to avoid recomputing it from the live DOM.
    */
-  updateTabVisibility(tabElements, isVisible) {
+  updateTabVisibility(
+    tabElements,
+    isVisible,
+    totalTabsCount = this.getTabsFromSlot().length
+  ) {
     tabElements.forEach((tab, index) => {
-      const tabsCount = this.getTabsFromSlot().length;
       // @ts-ignore
       tab.style.setProperty(
         'order',
-        isVisible ? index + 1 : tabsCount - tabElements.length + index + 1
+        isVisible ? index + 1 : totalTabsCount - tabElements.length + index + 1
       );
       // @ts-ignore
       tab.style.setProperty('visibility', isVisible ? 'visible' : 'hidden');
     });
   }
 
-  /**
-   * Indicates whether the tabs are causing an overflow.
-   * @returns {boolean}
-   */
-  get isOverflow() {
-    return this.slotContentWidth > this.containerWidth;
-  }
-
   /**
    * Returns the tab bar container element.
    * @returns {Element}
@@ -151,80 +264,18 @@ export default class QuanticTabBar extends LightningElement {
   }
 
   /**
-   * Returns the container's width.
-   * @returns {number}
-   */
-  get containerWidth() {
-    return getAbsoluteWidth(this.container);
-  }
-
-  /**
-   * returns the width of the content of the slot.
+   * Computes the total rendered width of the given tab elements.
+   * @param {Array<Element>} tabElements
    * @returns {number}
    */
-  get slotContentWidth() {
-    return this.getTabsFromSlot().reduce(
-      (total, el) => total + getAbsoluteWidth(el),
+  computeSlotContentWidth(tabElements) {
+    return tabElements.reduce(
+      (total, el) =>
+        total + Math.ceil(this._layoutRects.tabs.get(el)?.width ?? 0),
       0
     );
   }
 
-  /**
-   * returns the width of the more button.
-   * @returns {number}
-   */
-  get moreButtonWidth() {
-    return this.moreButton ? getAbsoluteWidth(this.moreButton) : 0;
-  }
-
-  /**
-   * returns the width of the currently selected tab.
-   * @returns {number}
-   */
-  get selectedTabWidth() {
-    return getAbsoluteWidth(this.selectedTab);
-  }
-
-  /**
-   * Returns the overflowing tabs.
-   * We compare the right position of each tab to the right position of the tab container to find the tabs that overflow.
-   * We include in our calculations the minimum width needed to display the elements that should always be displayed, namely the More button and the currently selected tab.
-   * @returns {Array<Element>}
-   */
-  get overflowingTabs() {
-    const containerRelativeRightPosition =
-      this.container.getBoundingClientRect().right;
-    const selectedTabRelativeRightPosition =
-      this.selectedTab?.getBoundingClientRect().right;
-
-    return this.getTabsFromSlot().filter((element) => {
-      const tabPositionedBeforeSelectedTab =
-        selectedTabRelativeRightPosition >
-        element.getBoundingClientRect().right;
-      const minimumWidthNeeded = tabPositionedBeforeSelectedTab
-        ? this.moreButtonWidth + this.selectedTabWidth
-        : this.moreButtonWidth;
-      const rightPositionLimit = !this.isOverflow
-        ? containerRelativeRightPosition
-        : containerRelativeRightPosition - minimumWidthNeeded;
-      return (
-        element.getBoundingClientRect().right > rightPositionLimit &&
-        // @ts-ignore
-        !element.isActive
-      );
-    });
-  }
-
-  /**
-   * Returns the displayed tabs.
-   * @returns {Array<Element>}
-   */
-  get displayedTabs() {
-    return this.getTabsFromSlot().filter(
-      (el) => !this.overflowingTabs.includes(el)
-    );
-  }
-
   /**
    * Returns the CSS classes of the dropdown list.
    * @returns {string}
@@ -266,18 +317,9 @@ export default class QuanticTabBar extends LightningElement {
    * @returns {string}
    */
   get moreButtonLabel() {
-    this.updateMoreButtonState();
     return this.expandedMoreButton ? this.labels.more : '';
   }
 
-  /**
-   * Returns the last visible tab element.
-   * @returns {Element}
-   */
-  get lastVisibleTab() {
-    return this.displayedTabs[this.displayedTabs.length - 1];
-  }
-
   /**
    * Returns the currently selected tab element.
    * @returns {Element}
@@ -287,25 +329,6 @@ export default class QuanticTabBar extends LightningElement {
     return this.getTabsFromSlot().find((el) => el.isActive);
   }
 
-  /**
-   * Returns the tab bar list container element.
-   * @returns {Element}
-   */
-  get tabBarListContainer() {
-    return this.template.querySelector('.tab-bar_list-container');
-  }
-
-  /**
-   * Returns the right position of the last visible tab.
-   * @returns {number}
-   */
-  get lastVisibleTabRightPosition() {
-    return (
-      this.lastVisibleTab.getBoundingClientRect().right -
-      this.tabBarListContainer.getBoundingClientRect().left
-    );
-  }
-
   /**
    * Returns the CSS classes of the tab bar container.
    * @returns {string}
@@ -343,7 +366,7 @@ export default class QuanticTabBar extends LightningElement {
     event.stopPropagation();
     const targetValue = event.currentTarget.getAttribute('data-value');
     const targetLabel = event.currentTarget.getAttribute('data-label');
-    const clickedtab = this.overflowingTabs.find(
+    const clickedtab = this._overflowingTabs.find(
       // @ts-ignore
       (tab) => tab.expression === targetValue && tab.label === targetLabel
     );