@ni/nimble-components 18.7.0 → 18.8.1

package/dist/all-components-bundle.js

@@ -16807,9 +16807,9 @@
         return `${prefix}${uniqueIdCounter++}`;
     }
 
-    const template$k = html `<slot></slot>`;
+    const template$l = html `<slot></slot>`;
 
-    const styles$J = css `
+    const styles$K = css `
     :host {
         display: contents;
     }
@@ -16865,8 +16865,8 @@
     ], ThemeProvider.prototype, "theme", void 0);
     const nimbleDesignSystemProvider = ThemeProvider.compose({
         baseName: 'theme-provider',
-        styles: styles$J,
-        template: template$k
+        styles: styles$K,
+        template: template$l
     });
     DesignSystem.getOrCreate()
         .withPrefix('nimble')
@@ -17050,7 +17050,7 @@
         }
     }
 
-    const styles$I = css `
+    const styles$J = css `
     ${display('inline')}
 
     :host {
@@ -17133,7 +17133,7 @@
 `;
 
     // prettier-ignore
-    const template$j = (context, definition) => html `
+    const template$k = (context, definition) => html `
     <a
         class="control"
         part="control"
@@ -17206,8 +17206,8 @@
     const nimbleAnchor = Anchor.compose({
         baseName: 'anchor',
         baseClass: Anchor$1,
-        template: template$j,
-        styles: styles$I,
+        template: template$k,
+        styles: styles$J,
         shadowOptions: {
             delegatesFocus: true
         }
@@ -17293,7 +17293,7 @@
         return new MultivaluePropertyStyleSheetBehavior('appearance', value, styles);
     }
 
-    const styles$H = css `
+    const styles$I = css `
     ${display('inline-flex')}
 
     :host {
@@ -17574,8 +17574,8 @@
             }
         `));
 
-    const styles$G = css `
-    ${styles$H}
+    const styles$H = css `
+    ${styles$I}
     ${buttonAppearanceVariantStyles}
 
     .control {
@@ -17583,7 +17583,7 @@
     }
 `;
 
-    const template$i = (context, definition) => html `
+    const template$j = (context, definition) => html `
     <a
         class="control"
         part="control"
@@ -17665,8 +17665,8 @@
     ], AnchorButton.prototype, "disabled", void 0);
     const nimbleAnchorButton = AnchorButton.compose({
         baseName: 'anchor-button',
-        template: template$i,
-        styles: styles$G,
+        template: template$j,
+        styles: styles$H,
         shadowOptions: {
             delegatesFocus: true
         }
@@ -17674,6 +17674,214 @@
     DesignSystem.getOrCreate().withPrefix('nimble').register(nimbleAnchorButton());
     DesignSystem.tagFor(AnchorButton);
 
+    const styles$G = css `
+    ${display('grid')}
+
+    :host {
+        font: ${bodyFont};
+    }
+
+    :host(:focus-within:not(:active)) {
+        outline: 2px solid ${borderHoverColor};
+        outline-offset: -2px;
+    }
+
+    :host(:hover) {
+        background: ${fillHoverColor};
+    }
+
+    :host(:active) {
+        background: ${fillSelectedColor};
+    }
+
+    :host([disabled]) {
+        background: transparent;
+    }
+
+    a {
+        display: grid;
+        contain: layout;
+        overflow: visible;
+        box-sizing: border-box;
+        height: ${controlHeight};
+        grid-template-columns: 1fr;
+        column-gap: 8px;
+        grid-template-rows: 1fr;
+        justify-items: start;
+        align-items: center;
+        margin: 0 0;
+        white-space: nowrap;
+        color: ${bodyFontColor};
+        fill: currentcolor;
+        cursor: pointer;
+        text-decoration: none;
+        outline: none;
+    }
+
+    :host([disabled]) a {
+        color: ${bodyDisabledFontColor};
+        fill: currentcolor;
+        cursor: default;
+    }
+
+    :host(.indent-1) a {
+        grid-template-columns: ${iconSize} 1fr;
+    }
+
+    [part='start'] {
+        display: contents;
+    }
+
+    slot[name='start']::slotted(*) {
+        fill: currentcolor;
+        width: ${iconSize};
+        height: ${iconSize};
+    }
+
+    :host(.indent-1) .start {
+        grid-column: 1;
+    }
+
+    :host(.indent-1) .content {
+        grid-column: 2;
+    }
+
+    [part='end'] {
+        display: none;
+    }
+`;
+
+    const template$i = (context, definition) => html `
+    <template
+        role="menuitem"
+        class="${x => (typeof x.startColumnCount === 'number'
+    ? `indent-${x.startColumnCount}`
+    : '')}"
+        aria-disabled="${x => x.disabled}"
+    >
+        <a
+            ${ref('anchor')}
+            download="${x => x.download}"
+            href=${x => (x.disabled ? null : x.href)}
+            hreflang="${x => x.hreflang}"
+            ping="${x => x.ping}"
+            referrerpolicy="${x => x.referrerpolicy}"
+            rel="${x => x.rel}"
+            target="${x => x.target}"
+            type="${x => x.type}"
+            @click="${(x, c) => x.clickHandler(c.event)}"
+            @keydown="${(x, c) => x.keydownHandler(c.event)}"
+        >
+            ${startSlotTemplate(context, definition)}
+            <span class="content" part="content">
+                <slot></slot>
+            </span>
+            ${endSlotTemplate(context, definition)}
+        </a>
+    </template>
+`;
+
+    /**
+     * A nimble-styled anchor menu-item
+     */
+    class AnchorMenuItem extends AnchorBase {
+        constructor() {
+            super(...arguments);
+            this.disabled = false;
+            /**
+             * There is an assumption that this component is styled using a grid display, and this property
+             * controls which grid column contains the menu item text (i.e. the indentation of the text).
+             * The parent menu sets this value on all its child menu items so their indentations align.
+             * @internal
+             */
+            this.startColumnCount = 0;
+        }
+        // The following two handlers are workarounds for issues with anchor menu items in submenus.
+        // Events can bubble up the DOM and get handled by the menu item in the parent menu. When that happens,
+        // the menu item's handlers (FAST) return false and prevent the default action, i.e. navigation.
+        // FAST has this issue about supporting links in menus: https://github.com/microsoft/fast/issues/5415
+        /**
+         * @internal
+         */
+        clickHandler(e) {
+            e.stopImmediatePropagation();
+            return true;
+        }
+        /**
+         * @internal
+         */
+        keydownHandler(e) {
+            if (e.defaultPrevented) {
+                return false;
+            }
+            switch (e.key) {
+                case keyEnter:
+                    e.stopImmediatePropagation();
+                    break;
+            }
+            return true;
+        }
+        // The FAST Menu manages the `tabindex` of its menu items. Because of a bug in Chromium
+        // (https://bugs.chromium.org/p/chromium/issues/detail?id=1346606), setting the tabindex on an element
+        // with `delegatesFocus=true` causes the element to lose focus. This causes the menu to close, preventing
+        // arrowing through the items or navigating to the url. As a workaround, we intercept attempts to
+        // set the tabindex on the host and instead set it on the inner anchor.
+        // The referenced Chromium bug is actually fixed, but it hasn't been pulled into Edge yet (it is in
+        // Chrome). Issue https://github.com/ni/nimble/issues/1118 tracks removal of this workaround.
+        setAttribute(qualifiedName, value) {
+            if (qualifiedName === 'tabindex') {
+                this.anchor.setAttribute(qualifiedName, value);
+            }
+            else {
+                super.setAttribute(qualifiedName, value);
+            }
+        }
+        // This is part of the bug workaround described above (in setAttribute)
+        get tabIndex() {
+            return this.anchor.tabIndex;
+        }
+        // This is part of the bug workaround described above (in setAttribute)
+        set tabIndex(value) {
+            this.anchor.tabIndex = value;
+        }
+    }
+    __decorate$1([
+        attr({ mode: 'boolean' })
+    ], AnchorMenuItem.prototype, "disabled", void 0);
+    __decorate$1([
+        observable
+    ], AnchorMenuItem.prototype, "anchor", void 0);
+    __decorate$1([
+        observable
+    ], AnchorMenuItem.prototype, "startColumnCount", void 0);
+    const nimbleAnchorMenuItem = AnchorMenuItem.compose({
+        baseName: 'anchor-menu-item',
+        template: template$i,
+        styles: styles$G,
+        shadowOptions: {
+            delegatesFocus: true
+        }
+    });
+    applyMixins(AnchorMenuItem, StartEnd);
+    DesignSystem.getOrCreate()
+        .withPrefix('nimble')
+        .register(nimbleAnchorMenuItem());
+    DesignSystem.tagFor(AnchorMenuItem);
+    // This is a workaround for the fact that FAST's menu uses `instanceof MenuItem`
+    // in their logic for indenting menu items. Since our AnchorMenuItem derives from
+    // AnchorBase and not FAST's MenuItem, we need to change their MenuItem's definition
+    // of `hasInstance` so that it includes our AnchorMenuItem, too.
+    //
+    // If/when we change FAST to test for the presence of `startColumnCount` instead
+    // of using `instanceof MenuItem`, we can remove this workaround. Here is the
+    // PR into FAST: https://github.com/microsoft/fast/pull/6667
+    const originalInstanceOf = MenuItem$1[Symbol.hasInstance].bind(MenuItem$1);
+    Object.defineProperty(MenuItem$1, Symbol.hasInstance, {
+        value(instance) {
+            return (originalInstanceOf(instance) || instance instanceof AnchorMenuItem);
+        }
+    });
+
     const styles$F = css `
     ${display('inline-flex')}
 
@@ -18322,7 +18530,7 @@
         `));
 
     const styles$A = css `
-    ${styles$H}
+    ${styles$I}
     ${buttonAppearanceVariantStyles}
 `;
 
@@ -19664,7 +19872,7 @@
     DesignSystem.tagFor(Checkbox);
 
     const styles$u = css `
-    ${styles$H}
+    ${styles$I}
 
     .control[aria-pressed='true'] {
         background-color: ${fillSelectedColor};
@@ -27476,7 +27684,6 @@
 
     .sort-indicator {
         padding: 0px calc(${standardPadding} / 2);
-        width: ${standardPadding};
     }
 `;
 
@@ -27485,14 +27692,12 @@
     <template role="columnheader" aria-sort="${x => x.ariaSort}">
         <slot></slot>
 
-        <span class="sort-indicator" aria-hidden="true">
-            ${when(x => x.sortDirection === TableColumnSortDirection.ascending, html `
-                <${iconArrowUpTag}></${iconArrowUpTag}>
-            `)}
-            ${when(x => x.sortDirection === TableColumnSortDirection.descending, html `
-                <${iconArrowDownTag}></${iconArrowDownTag}>
-            `)}
-        </span>
+        ${when(x => x.sortDirection === TableColumnSortDirection.ascending, html `
+            <${iconArrowUpTag} class="sort-indicator" aria-hidden="true"></${iconArrowUpTag}>
+        `)}
+        ${when(x => x.sortDirection === TableColumnSortDirection.descending, html `
+            <${iconArrowDownTag} class="sort-indicator" aria-hidden="true"></${iconArrowDownTag}>
+        `)}
     </template>
 `;