@lion/ui 0.19.1 → 0.20.0-menu-system.0

package/components/menu/src/LionMenu.js

@@ -0,0 +1,250 @@
+/* eslint-disable max-classes-per-file */
+/* eslint-disable import/no-extraneous-dependencies */
+import { LitElement } from 'lit';
+// import { dedupeMixin } from '@open-wc/dedupe-mixin';
+import { MultiLevelListMixin } from './MultiLevelListMixin.js';
+import { setChecked, toggleChecked } from './utils/listItemInteractions.js';
+
+// /**
+//  * @param {Element} element
+//  */
+// function getContentHeight(element) {
+//   return `${element.getBoundingClientRect().height}px`;
+// }
+
+// /**
+//  * Calculate total content height after collapsible opens
+//  * @param {HTMLElement} contentNode content node
+//  * @private
+//  */
+// async function calculateHeight(contentNode) {
+//   contentNode.style.setProperty('max-height', '');
+//   await new Promise(resolve => requestAnimationFrame(() => resolve(undefined)));
+//   return getContentHeight(contentNode); // Expected height i.e. actual size once collapsed after animation
+// }
+
+// /**
+//  * @type {AnimateMixin}
+//  */
+// const AnimateMixinImplementation = superclass =>
+//   class AnimateMixin extends superclass {
+//     constructor() {
+//       super();
+//       this.__handleAnimateComplete = this.__handleAnimateComplete.bind(this);
+//     }
+
+//     connectedCallback() {
+//       super.connectedCallback();
+//       this._contentNode.style.setProperty('transition', 'max-height 0.35s, opacity 0.35s');
+//     }
+
+//     /**
+//      * Trigger show animation and wait for transition to be finished.
+//      * @param {Object} options - element node and its options
+//      * @override
+//      */
+//     async _showAnimation(cfg) {
+//       super._showAnimation(cfg);
+
+//       const { contentNode } = cfg;
+//       const expectedHeight = await calculateHeight(contentNode);
+//       contentNode.style.setProperty('overflow', 'hidden');
+//       contentNode.style.setProperty('opacity', '1');
+//       contentNode.style.setProperty('max-height', '0');
+//       await new Promise(resolve => requestAnimationFrame(() => resolve(undefined)));
+//       contentNode.style.setProperty('max-height', expectedHeight);
+//       await this._animateComplete;
+//       ['opacity', 'padding', 'max-height', 'overflow'].map(prop =>
+//         contentNode.style.removeProperty(prop),
+//       );
+//     }
+
+//     /**
+//      * Trigger hide animation and wait for transition to be finished.
+//      * @param {Object} options - element node and its options
+//      * @override
+//      */
+//     async _hideAnimation(cfg) {
+//       super._hideAnimation(cfg);
+
+//       const { contentNode } = cfg;
+//       if (getContentHeight(contentNode) === '0px') {
+//         return;
+//       }
+//       const expectedHeight = await calculateHeight(contentNode);
+
+//       contentNode.style.setProperty('overflow', 'hidden');
+//       contentNode.style.setProperty('max-height', expectedHeight);
+//       await new Promise(resolve => requestAnimationFrame(() => resolve()));
+//       ['opacity', 'padding', 'max-height'].map(prop => contentNode.style.setProperty(prop, 0));
+
+//       await this._animateComplete;
+//     }
+//   };
+// export const AnimateMixin = dedupeMixin(AnimateMixinImplementation);
+
+/**
+ *
+ * Api explanation:
+ * - No [slot="invoker"], because adding it inside the lion-menu would:
+ *  - make api more verbose, less aligned with common html examples (examples from wai-aria etc.)
+ *  - break Popper positioning (you would need to specify parent menuitem as reference node)
+ *  - put click surface inside menuitem padding (you would need to correct with negative margin :( ...)
+ * - No [slot="content"], because for a "right click menu" it would be obsolete
+ *
+ * @example menu button (VsCode action menu right top of screen)
+ * <lion-menu>
+ *   <button slot="invoker" title="More actions">...</button>
+ *   <div role="menuitem">Show opened editors</div>
+ *   <div role="separator"></div>
+ *   <div role="menuitem">Close All</div>
+ * </lion-menu>
+ *
+ * @example as menubar (VsCode main menu)
+ * <lion-menu bar>
+ *   <div>
+ *     <div role="menuitem">File</div>
+ *     <lion-menu>
+ *       <div role="menuitem">New File</div>
+ *       <div role="separator"></div>
+ *       <div>
+ *         <div role="menuitem">Open Recent</div>
+ *         <lion-menu>
+ *           More...
+ *         </lion-menu>
+ *       </div>
+ *     </lion-menu>
+ *   </div>
+ *   <div>
+ *     <div role="menuitem">View</div>
+ *     <lion-menu>
+ *       <div role="menuitemcheckbox">Show Minimap</div>
+ *     </lion-menu>
+ *   </div>
+ * </lion-menu>
+ *
+ *
+ * @example as context menu (VsCode right click menu)
+ * <lion-menu>
+ *   <div role="menuitem">Go to Definition</div>
+ *   <div role="menuitem">Go to Type Definition</div>
+ *   <div>
+ *     <div role="menuitem">Peek</div>
+ *     <lion-menu>
+ *       <div role="menuitem">Peek Call Hierarchy</div>
+ *       <div role="separator"></div>
+ *       <div role="menuitem">Peek Definition</div>
+ *     </lion-menu>
+ *   </div>
+ *   <div role="separator"></div>
+ *   <div role="menuitem">Find all References</div>
+ * </lion-menu>
+ */
+// @ts-ignore - _listRole property type compatibility
+export class LionMenu extends MultiLevelListMixin(LitElement) {
+  static get properties() {
+    return {
+      /**
+       * Enable bar to use [role="menubar"] and horizontal navigation
+       */
+      bar: { type: Boolean },
+    };
+  }
+
+  /**
+   * Allows groups within one level. In case we deal with menuitemcheckbox,
+   * we treat it as multiple choice group
+   * @override InteractiveListMixin
+   * @param {number} index
+   */
+  setCheckedIndex(index) {
+    const item = this.listItems[index];
+    if (!item) return;
+
+    // @ts-ignore - InteractiveListItemRole type
+    const role = /** @type {InteractiveListItemRole} */ (item.getAttribute('role'));
+    let listItemsWithinGroup = this.listItems;
+    let multiple = this.multipleChoice;
+    if (role === 'menuitemradio' || role === 'menuitemcheckbox') {
+      /**
+       * If index = 3 (menuitemradio 'Red'), closest group will be div[role=group]
+       * @example
+       * <interactive-list role="menu">
+       *   <div role="menuitemcheckbox" aria-checked="true">Bold</div>
+       *   <div role="menuitemcheckbox" aria-checked="true">Italic</div>
+       *   <div role="separator"></div>
+       *   <div role="group" aria-label="Text Color">
+       *     <div role="menuitemradio" aria-checked="false">Blue</div>
+       *     <div role="menuitemradio" aria-checked="true">Red</div>
+       *     <div role="menuitemradio" aria-checked="false">Green</div>
+       *   </div>
+       * </interactive-list>
+       */
+      const closestGroup = item.closest('[role="group"]');
+      const group = closestGroup && this.contains(closestGroup) ? closestGroup : this;
+      listItemsWithinGroup = this.listItems.filter(listItem => group.contains(listItem));
+      multiple = role === 'menuitemcheckbox';
+    }
+
+    if (!multiple) {
+      // Uncheck all within group
+      listItemsWithinGroup.forEach(listItem => {
+        setChecked(listItem, true);
+      });
+      setChecked(this.listItems[index]);
+    } else {
+      toggleChecked(this.listItems[index]);
+    }
+  }
+
+  constructor() {
+    super();
+
+    this.bar = false;
+    /** @configure MultiLevelListMixin */
+    this.behaveAsAccordion = true;
+    /** @configure InteractiveListMixin */
+    this._listRole = 'menu';
+    /** @configure InteractiveListMixin */
+    this._activateOnTypedChars = true;
+    /** @configure DisclosureMixin */
+    this.invokerInteraction = 'click';
+  }
+
+  /**
+   * @param {import('lit-element').PropertyValues } changedProperties
+   */
+  firstUpdated(changedProperties) {
+    super.firstUpdated(changedProperties);
+
+    if (this._activeMode === 'tabbable-disclosure') {
+      this._listRole = 'list';
+    }
+
+    this._listNode.setAttribute('role', this._listRole);
+
+    // Allow generic attr for functional styling
+    this.dataset.menu = '';
+  }
+
+  /**
+   * @param {import('lit-element').PropertyValues } changedProperties
+   */
+  updated(changedProperties) {
+    super.updated(changedProperties);
+
+    if (changedProperties.has('bar')) {
+      if (this.bar) {
+        this.orientation = 'horizontal';
+        if (this._listRole === 'menu') {
+          this._listRole = 'menubar';
+        }
+      } else {
+        this.orientation = 'vertical';
+        if (this._listRole === 'menubar') {
+          this._listRole = 'menu';
+        }
+      }
+    }
+  }
+}

package/components/menu/src/LionMenuHybrid.js

@@ -0,0 +1,46 @@
+import { LionMenuOverlay } from './LionMenuOverlay.js';
+
+/**
+ * LionMenuHybrid allows to choose whether the menu opens as overlay or as disclosure.
+ */
+
+// TODO: remove
+// @ts-ignore - class extension properties mismatch
+export class LionMenuHybrid extends LionMenuOverlay {
+  static get properties() {
+    return {
+      openableMode: { type: String, attribute: 'openable-mode' },
+    };
+  }
+
+  constructor() {
+    super();
+
+    // By default, we go for disclosure behavior
+    // a TODO: in the future, bring disclosure behavior to a controller (and therefore directive).
+    // Take inspiration from VisibilityToggleCtrl of portal elements
+    /**
+     * Terminology aligned with https://open-ui.org/components/openable.explainer/
+     * @type {'disclosure'|'overlay'}
+     */
+    this.openableMode = 'disclosure';
+
+    this._shouldSetupOverlay = false;
+  }
+
+  /**
+   * @param {import('lit').PropertyValues} changedProperties
+   */
+  updated(changedProperties) {
+    super.updated(changedProperties);
+
+    if (changedProperties.has('openableMode')) {
+      if (this.openableMode === 'overlay') {
+        this._shouldSetupOverlay = true;
+        this._setupOverlayCtrl();
+      } else {
+        this._teardownOverlayCtrl();
+      }
+    }
+  }
+}

package/components/menu/src/LionMenuOverlay.js

@@ -0,0 +1,155 @@
+/* eslint-disable max-classes-per-file */
+/* eslint-disable import/no-extraneous-dependencies */
+import { html } from 'lit';
+import { dedupeMixin } from '@open-wc/dedupe-mixin';
+
+import { OverlayMixin, withDropdownConfig } from '@lion/ui/overlays.js';
+import { LionMenu } from './LionMenu.js';
+import { InteractiveListMixin } from './InteractiveListMixin.js';
+
+/**
+ * Handles integration of InteractiveListMixin and OverlayMixin
+ * Will be used by:
+ * - LionMenuOverlay
+ * - LionCombobox
+ * - LionSelectRich
+ *
+ * @param {import('@open-wc/dedupe-mixin').Constructor<import('lit').LitElement>} superclass
+ */
+const OverlayWithListInvokerMixinImplementation = superclass =>
+  class OverlayWithListInvokerMixin extends OverlayMixin(InteractiveListMixin(superclass)) {
+    // constructor() {
+    //   super();
+
+    //   /** @configure DisclosureMixin */
+    //   this.handleFocus = true;
+    // }
+
+    // _onOverlayBeforeShow = () => {};
+
+    _onOverlayShow = () => {
+      if (this.checkedIndex != null) {
+        // @ts-ignore - activeIndex can be number or array
+        this.activeIndex = this.checkedIndex;
+      }
+      // this._listNode.focus();
+    };
+
+    // _onOverlayHide = () => {
+    //   console.log('hide', this._overlayInvokerNode);
+    //   this._overlayInvokerNode.focus();
+    // };
+
+    /**
+     * @enhance OverlayMixin
+     */
+    _setupOverlayCtrl() {
+      super._setupOverlayCtrl();
+
+      if (!this._overlayCtrl) return;
+
+      // this._overlayCtrl.addEventListener('before-show', this._onOverlayBeforeShow);
+      this._overlayCtrl.addEventListener('show', this._onOverlayShow);
+      // this._overlayCtrl.addEventListener('hide', this._onOverlayHide);
+    }
+
+    /**
+     * @enhance OverlayMixin
+     */
+    _teardownOverlayCtrl() {
+      super._teardownOverlayCtrl();
+
+      if (!this._overlayCtrl) return;
+
+      // this._overlayCtrl.removeEventListener('before-show', this._onOverlayBeforeShow);
+      this._overlayCtrl.removeEventListener('show', this._onOverlayShow);
+      // this._overlayCtrl.removeEventListener('hide', this._onOverlayHide);
+    }
+
+    /**
+     * make sure OverlayMixin gets the contentNode defined by DisclosureMixin
+     */
+    get _overlayContentNode() {
+      // @ts-ignore - _contentNode property
+      return this._contentNode;
+    }
+
+    /**
+     * make sure OverlayMixin gets the invokerNode defined by DisclosureMixin
+     */
+    get _overlayInvokerNode() {
+      // @ts-ignore - _invokerNode property
+      return this._invokerNode;
+    }
+  };
+export const OverlayWithListInvokerMixin = dedupeMixin(OverlayWithListInvokerMixinImplementation);
+
+export class LionMenuOverlay extends OverlayWithListInvokerMixin(LionMenu) {
+  // TODO: this was created 5 years ago, do we still need id="overlay-content-node-wrapper" after the "dialog refactor"?
+  render() {
+    return html`
+      <slot name="invoker"></slot>
+      <div id="overlay-content-node-wrapper">
+        <slot name="list"></slot>
+      </div>
+      <slot id="list-items-outlet"></slot>
+    `;
+  }
+
+  // @ts-ignore - overlay config return type
+  _defineOverlayConfig() {
+    // @ts-ignore - parentList property
+    const { parentList } = this;
+    let placement = 'bottom-start';
+    if (parentList?.orientation !== 'horizontal') {
+      placement = 'right-start';
+    }
+
+    const menuConfig = {};
+    // if (this._activeMode === 'activedescendant') {
+    //   menuConfig.elementToFocusAfterHide = this._listNode;
+    // }
+
+    const dropdownCfg = withDropdownConfig();
+
+    return {
+      ...dropdownCfg,
+      hidesOnEsc: true,
+      ...menuConfig,
+      popperConfig: {
+        ...dropdownCfg.popperConfig,
+        placement,
+        strategy: 'absolute',
+        modifiers: [
+          {
+            name: 'offset',
+            enabled: true,
+            options: {
+              offset: [0, 0],
+            },
+          },
+        ],
+      },
+    };
+  }
+
+  /**
+   * @enhance InteractiveListMixin
+   */
+  /**
+   * @param {*} ev
+   */
+  _onListKeyUp(ev) {
+    super._onListKeyUp(ev);
+
+    const { key } = ev;
+
+    switch (key) {
+      case 'Escape':
+        // We need to stop here, or else we affect parent menu (handled by OverlayController)
+        ev.stopPropagation();
+        break;
+      /* no default */
+    }
+  }
+}

package/components/menu/src/LionSpinbutton.js

@@ -0,0 +1,213 @@
+/* eslint-disable import/no-extraneous-dependencies */
+import { LitElement, html, css } from 'lit';
+
+export class LionSpinbutton extends LitElement {
+  static get properties() {
+    return {
+      now: { type: Number },
+      min: { type: Number },
+      max: { type: Number },
+      unit: { type: String },
+      step: { type: Number },
+    };
+  }
+
+  get _decreaseNode() {
+    return this.shadowRoot?.querySelector('[data-ref=decrease]');
+  }
+
+  get _increaseNode() {
+    return this.shadowRoot?.querySelector('[data-ref=increase]');
+  }
+
+  static get styles() {
+    return css`
+      :host {
+        display: flex;
+        align-items: center;
+        cursor: default;
+      }
+    `;
+  }
+
+  render() {
+    return this._mainTemplate({ ariaText: this._getAriaText() });
+  }
+
+  /**
+   * @param {{ ariaText: string }} props
+   */
+  _mainTemplate({ ariaText }) {
+    return html`${ariaText}${this._decreaseBtnTemplate()}${this._increaseBtnTemplate()}`;
+  }
+
+  // eslint-disable-next-line class-methods-use-this
+  _decreaseBtnTemplate() {
+    return html`<span data-ref="decrease">▼</span>`;
+  }
+
+  // eslint-disable-next-line class-methods-use-this
+  _increaseBtnTemplate() {
+    return html`<span data-ref="increase">▲</span>`;
+  }
+
+  constructor() {
+    super();
+    this.unit = '';
+
+    this.step = 1;
+
+    this.__decrease = this.__decrease.bind(this);
+    this.__increase = this.__increase.bind(this);
+  }
+
+  connectedCallback() {
+    super.connectedCallback();
+
+    if (this.__isSetup) {
+      return;
+    }
+
+    this.setAttribute('tabindex', '0');
+    this.__isSetup = true;
+  }
+
+  /**
+   * @param {import('lit').PropertyValues} changedProperties
+   */
+  firstUpdated(changedProperties) {
+    super.firstUpdated(changedProperties);
+
+    // @ts-ignore - event listener type compatibility
+    this._decreaseNode?.addEventListener('mousedown', this.__decrease);
+    // @ts-ignore - event listener type compatibility
+    this._increaseNode?.addEventListener('mousedown', this.__increase);
+    this.addEventListener('keydown', this._onKeydown);
+
+    this.addEventListener('mousedown', () => {
+      this.__isMousedown = true;
+    });
+    this.addEventListener('mouseup', () => {
+      this.__isMousedown = false;
+    });
+  }
+
+  /**
+   * @param {import('lit').PropertyValues} changedProperties
+   */
+  updated(changedProperties) {
+    super.updated(changedProperties);
+
+    if (changedProperties.has('now')) {
+      this.setAttribute('aria-valuenow', `${this.now}`);
+    }
+
+    if (changedProperties.has('now') || changedProperties.has('unit')) {
+      this.setAttribute('aria-valuetext', this._getAriaText());
+    }
+
+    if (changedProperties.has('min')) {
+      // @ts-ignore - min property from properties
+      this.setAttribute('aria-valuemin', `${this.min}`);
+    }
+
+    if (changedProperties.has('max')) {
+      // @ts-ignore - max property from properties
+      this.setAttribute('aria-valuemax', `${this.max}`);
+    }
+  }
+
+  /**
+   * @overridable
+   */
+  _getAriaText() {
+    return `${this.now}${this.unit ? this.unit : ''}`;
+  }
+
+  /**
+   * @param {MouseEvent | null} ev
+   * @param {number} step
+   * @param {number} timeout
+   */
+  __decrease(ev, step = this.step, timeout = 100) {
+    // @ts-ignore - min property from properties
+    if (this.min !== undefined && this.now - 1 >= this.min) {
+      this.now -= 1; // TODO: support step/increment?
+    }
+    // @ts-ignore - ev can be null
+    if (ev.type === 'mousedown') {
+      setTimeout(() => {
+        if (this.__isMousedown) {
+          this.__decrease(ev, step, timeout / 1.3);
+        }
+      }, timeout);
+    }
+  }
+
+  /**
+   * @param {MouseEvent | null} ev
+   * @param {number} step
+   * @param {number} timeout
+   */
+  __increase(ev, step = this.step, timeout = 100) {
+    // @ts-ignore - max property from properties
+    if (this.max !== undefined && this.now + 1 <= this.max) {
+      this.now += 1; // TODO: support step/increment?
+    }
+    // @ts-ignore - ev can be null
+    if (ev.type === 'mousedown') {
+      setTimeout(() => {
+        if (this.__isMousedown) {
+          this.__increase(ev, step, timeout / 1.3);
+        }
+      }, timeout);
+    }
+  }
+
+  /**
+   * @param {KeyboardEvent} ev
+   */
+  _onKeydown(ev) {
+    const { key } = ev;
+
+    // eslint-disable-next-line default-case
+    switch (key) {
+      case 'ArrowDown':
+        ev.stopPropagation(); // In case we are inside a menu
+        ev.preventDefault(); // Prevent scroll
+        // @ts-ignore - passing null for ev parameter
+        this.__decrease();
+        break;
+      case 'ArrowUp':
+        ev.stopPropagation(); // In case we are inside a menu
+        ev.preventDefault(); // Prevent scroll
+        // @ts-ignore - passing null for ev parameter
+        this.__increase();
+        break;
+      case 'Home':
+        ev.stopPropagation(); // In case we are inside a menu
+        ev.preventDefault(); // Prevent scroll
+        // @ts-ignore - min property from properties
+        if (this.min !== undefined) {
+          // @ts-ignore
+          this.now = this.min;
+        }
+        break;
+      case 'End':
+        ev.stopPropagation(); // In case we are inside a menu
+        ev.preventDefault(); // Prevent scroll
+        // @ts-ignore - max property from properties
+        if (this.max !== undefined) {
+          // @ts-ignore
+          this.now = this.max;
+        }
+        break;
+      case 'PageDown':
+        this.__decrease(null, this.step * 2);
+        break;
+      case 'PageUp':
+        this.__increase(null, this.step * 2);
+        break;
+    }
+  }
+}