@hashicorp/design-system-components 2.13.0 → 2.14.1

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @hashicorp/design-system-components
 
+## 2.14.1
+
+### Patch Changes
+
+- [#1709](https://github.com/hashicorp/design-system/pull/1709) [`294dddfda`](https://github.com/hashicorp/design-system/commit/294dddfda8d8e174fd5dd7de5d6af01dd8405775) Thanks [@didoo](https://github.com/didoo)! - `Tabs` - Fixed issue with `@isSelected` dynamically changed within `#each` loops
+
+## 2.14.0
+
+### Minor Changes
+
+- [#1700](https://github.com/hashicorp/design-system/pull/1700) [`33d760fb8`](https://github.com/hashicorp/design-system/commit/33d760fb88d3945be8b50302a9bb7dce3ae221fe) Thanks [@didoo](https://github.com/didoo)! - `Pagination::Compact` - Added option to show "SizeSelector" element
+
+- [#1688](https://github.com/hashicorp/design-system/pull/1688) [`c842b6eb7`](https://github.com/hashicorp/design-system/commit/c842b6eb731d82146b0e1ad8b9f55930b58aba18) Thanks [@didoo](https://github.com/didoo)! - `Tabs` - Refactored logic for `Tabs` component + `Tab/Panel` sub-components to support more complex use cases:
+
+  - introduced `@selectedTabIndex` argument to control the "selected" tab from the consuming application, e.g. via query params (effort spearheaded by @MiniHeyd)
+  - fixed issue with nested tabs not initializing the "selected" indicator correctly
+  - fixed issue with dynamic tab content not updating the "selected" indicator correctly
+
 ## 2.13.0
 
 ### Minor Changes

package/addon/components/hds/pagination/compact/index.hbs

@@ -27,4 +27,13 @@
       @disabled={{@isDisabledNext}}
     />
   </nav>
+
+  {{#if this.showSizeSelector}}
+    <Hds::Pagination::SizeSelector
+      @pageSizes={{this.pageSizes}}
+      @label={{@sizeSelectorLabel}}
+      @selectedSize={{this.currentPageSize}}
+      @onChange={{this.onPageSizeChange}}
+    />
+  {{/if}}
 </div>

package/addon/components/hds/pagination/compact/index.js

@@ -8,9 +8,15 @@ import { action } from '@ember/object';
 import { assert } from '@ember/debug';
 import { inject as service } from '@ember/service';
 
+// for context about the decision to use these values, see:
+// https://hashicorp.slack.com/archives/C03A0N1QK8S/p1673546329082759
+export const DEFAULT_PAGE_SIZES = [10, 30, 50];
+
 export default class HdsPaginationCompactIndexComponent extends Component {
   @service router;
 
+  showSizeSelector = this.args.showSizeSelector ?? false; // if the "size selector" block is visible
+
   constructor() {
     super(...arguments);
 
@@ -56,6 +62,23 @@ export default class HdsPaginationCompactIndexComponent extends Component {
     return this.args.ariaLabel ?? 'Pagination';
   }
 
+  /**
+   * @param pageSizes
+   * @type {array of numbers}
+   * @description Set the page sizes users can select from.
+   * @default [10, 30, 50]
+   */
+  get pageSizes() {
+    let { pageSizes = DEFAULT_PAGE_SIZES } = this.args;
+
+    assert(
+      `pageSizes argument must be an array. Received: ${pageSizes}`,
+      Array.isArray(pageSizes) === true
+    );
+
+    return pageSizes;
+  }
+
   get routeQueryParams() {
     return this.router.currentRoute?.queryParams || {};
   }

package/addon/components/hds/tabs/index.hbs

@@ -3,7 +3,14 @@
   SPDX-License-Identifier: MPL-2.0
 }}
 {{! template-lint-disable no-invalid-role }}
-<div class="hds-tabs" {{did-insert this.didInsert}} ...attributes>
+<div
+  class="hds-tabs"
+  {{did-insert this.didInsert}}
+  {{did-update this.didUpdateSelectedTabIndex this.selectedTabIndex}}
+  {{did-update this.didUpdateSelectedTabId this.selectedTabId}}
+  {{did-update this.didUpdateParentVisibility @isParentVisible}}
+  ...attributes
+>
   <div class="hds-tabs__tablist-wrapper">
     <ul class="hds-tabs__tablist" role="tablist">
       {{yield
@@ -11,9 +18,9 @@
           Tab=(component
             "hds/tabs/tab"
             didInsertNode=this.didInsertTab
+            didUpdateNode=this.didUpdateTab
             willDestroyNode=this.willDestroyTab
             tabIds=this.tabIds
-            panelIds=this.panelIds
             selectedTabIndex=this.selectedTabIndex
             onClick=this.onClick
             onKeyUp=this.onKeyUp

package/addon/components/hds/tabs/index.js

@@ -7,45 +7,97 @@ import Component from '@glimmer/component';
 import { tracked } from '@glimmer/tracking';
 import { action } from '@ember/object';
 import { assert } from '@ember/debug';
-import { schedule } from '@ember/runloop';
+import { next, schedule } from '@ember/runloop';
 
 export default class HdsTabsIndexComponent extends Component {
   @tracked tabNodes = [];
   @tracked tabIds = [];
   @tracked panelNodes = [];
   @tracked panelIds = [];
-  @tracked selectedTabIndex;
+  @tracked _selectedTabIndex = this.args.selectedTabIndex ?? 0;
+  @tracked selectedTabId;
+  @tracked isControlled;
+
+  constructor() {
+    super(...arguments);
+
+    // this is to determine if the "selected" tab logic is controlled in the consumers' code or is maintained as an internal state
+    this.isControlled = this.args.selectedTabIndex !== undefined;
+  }
+
+  get selectedTabIndex() {
+    if (this.isControlled) {
+      return this.args.selectedTabIndex;
+    } else {
+      return this._selectedTabIndex;
+    }
+  }
+
+  set selectedTabIndex(value) {
+    if (this.isControlled) {
+      // noop
+    } else {
+      this._selectedTabIndex = value;
+    }
+  }
 
   @action
   didInsert() {
-    // default starting tab index
-    let initialTabIndex = 0;
-    let selectedCount = 0;
-
-    this.tabNodes.forEach((tabElement, index) => {
-      if (tabElement.hasAttribute('data-is-selected')) {
-        initialTabIndex = index;
-        selectedCount++;
-      }
+    assert(
+      'The number of Tabs must be equal to the number of Panels',
+      this.tabNodes.length === this.panelNodes.length
+    );
+
+    if (this.selectedTabId) {
+      this.selectedTabIndex = this.tabIds.indexOf(this.selectedTabId);
+    }
+
+    schedule('afterRender', () => {
+      this.setTabIndicator();
     });
-    this.selectedTabIndex = initialTabIndex;
+  }
 
+  @action
+  didUpdateSelectedTabIndex() {
     schedule('afterRender', () => {
-      this.setTabIndicator(initialTabIndex);
+      this.setTabIndicator();
     });
+  }
 
-    assert('Only one tab may use isSelected argument', selectedCount <= 1);
+  @action
+  didUpdateSelectedTabId() {
+    // if the selected tab is set dynamically (eg. in a `each` loop)
+    // the `Tab` nodes will be re-inserted/rendered, which means the `this.selectedTabId` variable changes
+    // but the parent `Tabs` component has already been rendered/inserted but doesn't re-render
+    // so the value of the `selectedTabIndex` is not updated, unless we trigger a recalculation
+    // using the `did-update` modifier that checks for changes in the `this.selectedTabId` variable
+    if (this.selectedTabId) {
+      this.selectedTabIndex = this.tabIds.indexOf(this.selectedTabId);
+    }
+  }
 
-    assert(
-      'The number of Tabs must be equal to the number of Panels',
-      this.tabNodes.length === this.panelNodes.length
-    );
+  @action
+  didUpdateParentVisibility() {
+    schedule('afterRender', () => {
+      this.setTabIndicator();
+    });
   }
 
   @action
-  didInsertTab(element) {
+  didInsertTab(element, isSelected) {
     this.tabNodes = [...this.tabNodes, element];
     this.tabIds = [...this.tabIds, element.id];
+    if (isSelected) {
+      this.selectedTabId = element.id;
+    }
+  }
+
+  @action
+  didUpdateTab(tabIndex, isSelected) {
+    if (isSelected) {
+      this.selectedTabIndex = tabIndex;
+    }
+    this.setTabIndicator();
   }
 
   @action
@@ -55,7 +107,7 @@ export default class HdsTabsIndexComponent extends Component {
   }
 
   @action
-  didInsertPanel(panelId, element) {
+  didInsertPanel(element, panelId) {
     this.panelNodes = [...this.panelNodes, element];
     this.panelIds = [...this.panelIds, panelId];
   }
@@ -67,40 +119,39 @@ export default class HdsTabsIndexComponent extends Component {
   }
 
   @action
-  onClick(tabIndex, event) {
+  onClick(event, tabIndex) {
     this.selectedTabIndex = tabIndex;
-    this.setTabIndicator(tabIndex);
-
-    // Scroll Tab into view if it's out of view
-    this.tabNodes[tabIndex].parentNode.scrollIntoView({
-      behavior: 'smooth',
-      block: 'nearest',
-      inline: 'nearest',
-    });
+    this.setTabIndicator();
 
     // invoke the callback function if it's provided as argument
     if (typeof this.args.onClickTab === 'function') {
-      this.args.onClickTab(event);
+      this.args.onClickTab(event, tabIndex);
     }
   }
 
   @action
-  onKeyUp(tabIndex, e) {
+  onKeyUp(tabIndex, event) {
     const leftArrow = 37;
     const rightArrow = 39;
     const enterKey = 13;
     const spaceKey = 32;
 
-    if (e.keyCode === rightArrow) {
+    if (event.keyCode === rightArrow) {
       const nextTabIndex = (tabIndex + 1) % this.tabIds.length;
-      this.focusTab(nextTabIndex, e);
-    } else if (e.keyCode === leftArrow) {
+      this.focusTab(nextTabIndex, event);
+    } else if (event.keyCode === leftArrow) {
       const prevTabIndex =
         (tabIndex + this.tabIds.length - 1) % this.tabIds.length;
-      this.focusTab(prevTabIndex, e);
-    } else if (e.keyCode === enterKey || e.keyCode === spaceKey) {
+      this.focusTab(prevTabIndex, event);
+    } else if (event.keyCode === enterKey || event.keyCode === spaceKey) {
       this.selectedTabIndex = tabIndex;
     }
+    // scroll selected tab into view (it may be out of view when activated using a keyboard with `prev/next`)
+    this.tabNodes[this.selectedTabIndex].parentNode.scrollIntoView({
+      behavior: 'smooth',
+      block: 'nearest',
+      inline: 'nearest',
+    });
   }
 
   // Focus tab for keyboard & mouse navigation:
@@ -114,15 +165,37 @@ export default class HdsTabsIndexComponent extends Component {
     this.panelNodes[tabIndex].focus();
   }
 
-  setTabIndicator(tabIndex) {
-    const tabElem = this.tabNodes[tabIndex];
-    const tabsParentElem = tabElem.closest('.hds-tabs');
-
-    const tabLeftPos = tabElem.parentNode.offsetLeft;
-    const tabWidth = tabElem.parentNode.offsetWidth;
-
-    // Set CSS custom properties for indicator
-    tabsParentElem.style.setProperty('--indicator-left-pos', tabLeftPos + 'px');
-    tabsParentElem.style.setProperty('--indicator-width', tabWidth + 'px');
+  setTabIndicator() {
+    next(() => {
+      const tabElem = this.tabNodes[this.selectedTabIndex];
+
+      if (tabElem) {
+        const tabsParentElem = tabElem.closest('.hds-tabs__tablist');
+
+        // this condition is `null` if any of the parents has `display: none`
+        if (tabElem.parentNode.offsetParent) {
+          const tabLeftPos = tabElem.parentNode.offsetLeft;
+          const tabWidth = tabElem.parentNode.offsetWidth;
+
+          // Set CSS custom properties for indicator
+          tabsParentElem.style.setProperty(
+            '--indicator-left-pos',
+            tabLeftPos + 'px'
+          );
+          tabsParentElem.style.setProperty(
+            '--indicator-width',
+            tabWidth + 'px'
+          );
+        }
+      } else {
+        assert(
+          `"Hds::Tabs" has tried to set the indicator for an element that doesn't exist (the value ${
+            this.selectedTabIndex
+          } of \`this.selectedTabIndex\` is out of bound for the array \`this.tabNodes\`, whose index range is [0-${
+            this.tabNodes.length - 1
+          }])`
+        );
+      }
+    });
   }
 }

package/addon/components/hds/tabs/panel.hbs

@@ -6,11 +6,11 @@
   class="hds-tabs__panel"
   ...attributes
   role="tabpanel"
-  aria-labelledby={{this.tabId}}
   id={{this.panelId}}
-  hidden={{not this.isSelected}}
+  hidden={{not this.isVisible}}
+  aria-labelledby={{this.coupledTabId}}
   {{did-insert this.didInsertNode}}
   {{will-destroy this.willDestroyNode}}
 >
-  {{yield}}
+  {{yield (hash isVisible=this.isVisible)}}
 </section>

package/addon/components/hds/tabs/panel.js

@@ -10,9 +10,8 @@ import { action } from '@ember/object';
 
 export default class HdsTabsIndexComponent extends Component {
   /**
-   * Generates a unique ID for the Panel
-   *
-   * @param panelId
+   * Generate a unique ID for the Panel
+   * @return {string}
    */
   panelId = 'panel-' + guidFor(this);
 
@@ -23,32 +22,40 @@ export default class HdsTabsIndexComponent extends Component {
       : undefined;
   }
 
-  get tabId() {
+  /**
+   * Check the condition if the panel is visible (because the coupled/associated tab is selected) or not
+   * @returns {boolean}
+   */
+  get isVisible() {
+    return this.nodeIndex === this.args.selectedTabIndex;
+  }
+
+  /**
+   * Get the ID of the tab coupled/associated with the panel (it's used by the `aria-labelledby` attribute)
+   * @returns string}
+   */
+  get coupledTabId() {
     return this.nodeIndex !== undefined
       ? this.args.tabIds[this.nodeIndex]
       : undefined;
   }
 
-  get isSelected() {
-    return this.nodeIndex === this.args.selectedTabIndex;
-  }
-
   @action
   didInsertNode(element) {
     let { didInsertNode } = this.args;
 
     if (typeof didInsertNode === 'function') {
       this.elementId = element.id;
-      didInsertNode(this.elementId, ...arguments);
+      didInsertNode(element, this.elementId);
     }
   }
 
   @action
-  willDestroyNode() {
+  willDestroyNode(element) {
     let { willDestroyNode } = this.args;
 
     if (typeof willDestroyNode === 'function') {
-      willDestroyNode(...arguments);
+      willDestroyNode(element);
     }
   }
 }

package/addon/components/hds/tabs/tab.hbs

@@ -11,8 +11,8 @@
     id={{this.tabId}}
     aria-selected={{if this.isSelected "true" "false"}}
     tabindex={{unless this.isSelected "-1"}}
-    data-is-selected={{this.isInitialTab}}
-    {{did-insert this.didInsertNode}}
+    {{did-insert this.didInsertNode @isSelected}}
+    {{did-update this.didUpdateNode @count @isSelected}}
     {{will-destroy this.willDestroyNode}}
     {{on "click" this.onClick}}
     {{on "keyup" this.onKeyUp}}

package/addon/components/hds/tabs/tab.js

@@ -10,9 +10,8 @@ import { action } from '@ember/object';
 
 export default class HdsTabsIndexComponent extends Component {
   /**
-   * Generates a unique ID for the Tab
-   *
-   * @param tabId
+   * Generate a unique ID for the Tab
+   * @return {string}
    */
   tabId = 'tab-' + guidFor(this);
 
@@ -21,17 +20,10 @@ export default class HdsTabsIndexComponent extends Component {
     return this.args.tabIds ? this.args.tabIds.indexOf(this.tabId) : undefined;
   }
 
-  get panelId() {
-    return this.nodeIndex !== undefined
-      ? this.args.panelIds[this.nodeIndex]
-      : undefined;
-  }
-
   /**
-   * @param isSelected
-   * @type {boolean}
+   * Determine if the tab is the selected tab
+   * @return {boolean}
    * @default false (1st tab is selected by default)
-   * @description Determines if the tab is the selected tab
    */
   get isSelected() {
     return (
@@ -40,46 +32,52 @@ export default class HdsTabsIndexComponent extends Component {
     );
   }
 
-  get isInitialTab() {
-    let { isSelected } = this.args;
-    return isSelected;
-  }
-
   @action
-  didInsertNode() {
+  didInsertNode(element, positional) {
     let { didInsertNode } = this.args;
 
+    const isSelected = positional[0];
+
     if (typeof didInsertNode === 'function') {
-      didInsertNode(...arguments);
+      didInsertNode(element, isSelected);
+    }
+  }
+
+  @action
+  didUpdateNode() {
+    let { didUpdateNode } = this.args;
+
+    if (typeof didUpdateNode === 'function') {
+      didUpdateNode(this.nodeIndex, this.args.isSelected);
     }
   }
 
   @action
-  willDestroyNode() {
+  willDestroyNode(element) {
     let { willDestroyNode } = this.args;
 
     if (typeof willDestroyNode === 'function') {
-      willDestroyNode(...arguments);
+      willDestroyNode(element);
     }
   }
 
   @action
-  onClick() {
+  onClick(event) {
     let { onClick } = this.args;
 
     if (typeof onClick === 'function') {
-      onClick(this.nodeIndex, ...arguments);
+      onClick(event, this.nodeIndex);
     } else {
       return false;
     }
   }
 
   @action
-  onKeyUp() {
+  onKeyUp(event) {
     let { onKeyUp } = this.args;
 
     if (typeof onKeyUp === 'function') {
-      onKeyUp(this.nodeIndex, ...arguments);
+      onKeyUp(this.nodeIndex, event);
     } else {
       return false;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hashicorp/design-system-components",
-  "version": "2.13.0",
+  "version": "2.14.1",
   "description": "Helios Design System Components",
   "keywords": [
     "hashicorp",