@veritree/ui 0.19.2 → 0.20.0-0

package/src/components/Listbox/VTListboxTrigger.vue

@@ -1,34 +1,20 @@
 <template>
   <button
+    :id="id"
+    :class="classComputed"
+    :disabled="disabled"
     :aria-expanded="expanded"
     :aria-haspopup="hasPopup"
-    :class="{
-      'Listbox-button': headless,
-      'flex w-full justify-between rounded-md border border-solid py-2 px-3':
-        !headless,
-      'border-gray-300 text-gray-500': !dark && !headless,
-      'border-white/70 text-white focus-visible:ring-2 focus-visible:ring-white':
-        dark && !headless,
-    }"
     type="button"
-    @click.stop.prevent="onClick"
-    @keydown.down.prevent="onKeyArrowDown"
-    @keydown.up.prevent="onKeyArrowUp"
+    @click.prevent="onClick"
+    @keydown.down.prevent="onKeyDownOrUp"
+    @keydown.up.prevent="onKeyDownOrUp"
     @keydown.esc.stop="onKeyEsc"
   >
-    <span
-      :class="{
-        'Listbox-button__text': headless,
-        'text-left': !headless,
-      }"
-    >
+    <span :class="[headless ? 'listbox-button__text' : 'truncate text-left']">
       <slot></slot>
     </span>
-    <span
-      :class="{
-        'Listbox-button__icon': headless,
-      }"
-    >
+    <span :class="[headless ? 'listbox-button__icon' : 'shrink-0']">
       <IconChevronDown
         class="transition-transform"
         :class="{ 'rotate-180': expanded }"
@@ -38,6 +24,10 @@
 </template>
 
 <script>
+import {
+  formControlMixin,
+  formControlStyleMixin,
+} from '../../../mixins/form-control';
 import { IconChevronDown } from '@veritree/icons';
 
 export default {
@@ -45,33 +35,29 @@ export default {
 
   components: { IconChevronDown },
 
-  inject: ['api'],
+  mixins: [formControlMixin, formControlStyleMixin],
 
-  props: {
-    headless: {
-      type: Boolean,
-      default: false,
-    },
-  },
+  inject: ['apiListbox'],
 
   data() {
     return {
+      name: 'listbox-trigger',
       expanded: false,
       hasPopup: false,
     };
   },
 
   computed: {
-    dark() {
-      return this.api().isDark;
+    id() {
+      return `listbox-trigger-${this.apiListbox().id}`;
     },
 
-    content() {
-      return this.api().content;
+    componentContent() {
+      return this.apiListbox().componentContent;
     },
 
     items() {
-      return this.api().items;
+      return this.apiListbox().items;
     },
 
     firstMenuItem() {
@@ -86,83 +72,86 @@ export default {
   mounted() {
     const trigger = {
       el: this.$el,
+      cancel: this.cancel,
       focus: this.focus,
-      onClick: this.onClick,
-      contract: this.contract,
+      id: this.id,
     };
 
-    this.api().registerTrigger(trigger);
+    this.apiListbox().registerTrigger(trigger);
   },
 
   methods: {
-    /**
-     * Shows content/menu if not already visible
-     */
-    showContent() {
+    init(e) {
+      if (!this.componentContent) {
+        return;
+      }
+
+      if (this.expanded) {
+        this.cancel();
+        return;
+      }
+
       this.expanded = true;
-      this.content.show();
-    },
 
-    focus() {
-      this.$el.focus();
+      // delay stop propagation to close other visible
+      // dropdowns and delay click event to control
+      // this dropdown visibility
+      setTimeout(() => e.stopImmediatePropagation(), 50);
+      setTimeout(() => this.showComponentContent(), 100);
     },
 
-    contract() {
-      if (!this.expanded) return;
+    cancel() {
+      if (!this.componentContent) {
+        return;
+      }
+
       this.expanded = false;
-    },
 
-    /**
-     * On click, do the following:
-     *
-     * 1. Toggle aria expanded attribute/state
-     * 2. Open the menu if it's closed
-     * 3. Close the menu if it's open
-     */
-    onClick() {
-      if (!this.content) return;
-      this.expanded ? this.content.hide() : this.showContent();
+      this.hideComponentContent();
     },
 
-    /**
-     * On key arrow down, do the following:
-     *
-     * 1. if the menu is not expanded, expand it and focus the first menu item
-     * 2. if the menu is expanded, focus the first menu item
-     */
-    onKeyArrowDown() {
-      if (!this.content) return;
+    focus() {
+      this.$el.focus();
+    },
 
-      this.showContent();
+    showComponentContent() {
+      this.componentContent.show();
+    },
 
-      this.$nextTick(() => {
-        this.firstMenuItem.focus();
-      });
+    hideComponentContent() {
+      this.componentContent.hide();
     },
 
-    /**
-     * On key arrow up, do the following:
-     *
-     * 1. if the menu is not expanded, expand it and focus the last menu item
-     * 2. if the menu is expanded, focus the last menu item
-     */
-    onKeyArrowUp() {
-      if (!this.content) return;
+    onClick(e) {
+      this.init(e);
+      this.$emit('click');
+    },
 
-      this.showContent();
+    onKeyDownOrUp(e) {
+      if (!this.expanded) {
+        this.$el.click(e);
+      }
 
+      const keyCode = e.code;
+      const listItemPosition =
+        keyCode === 'ArrowDown'
+          ? 'firstMenuItem'
+          : keyCode === 'ArrowUp'
+          ? 'lastMenuItem'
+          : null;
+
+      // settimeout here is delaying the focusing the element
+      // since it is not rendered yet. All items will only
+      // be available when the content is fully visible.
       this.$nextTick(() => {
-        this.lastMenuItem.focus();
+        setTimeout(() => this[listItemPosition].focus(), 100);
       });
     },
 
+    // change it to a better name or move the methods inside to another function
     onKeyEsc() {
-      if (!this.content) return;
-
-      if (this.expanded) {
-        this.toggleExpanded();
-        this.content.hide();
-      }
+      this.cancel();
+      this.focus();
     },
   },
 };

package/src/components/Popover/VTPopover.vue

@@ -2,43 +2,41 @@
   <div
     :id="id"
     class="relative"
-    :aria-haspopup="content ? 'true' : null"
-    :aria-controls="content ? content.id : null"
+    :aria-haspopup="componentContent ? 'true' : null"
+    :aria-controls="componentContent ? componentContent.id : null"
   >
     <slot></slot>
   </div>
 </template>
 
 <script>
-import { genId } from "../../utils/ids";
+import { floatingUiMixin } from '../../../mixins/floating-ui';
+import { genId } from '../../utils/ids';
 
 export default {
-  name: "VTPopover",
+  name: 'VTPopover',
+
+  mixins: [floatingUiMixin],
 
   provide() {
     return {
-      api: () => {
-        const { dark: isDark, headless: isHeadless, right: isRight } = this;
-        const { id, button, content } = this;
-
-        const registerButton = (button) => {
-          if (!button) return;
-          this.button = button;
+      apiPopover: () => {
+        const registerTrigger = (trigger) => {
+          if (!trigger) return;
+          this.componentTrigger = trigger;
         };
 
         const registerContent = (content) => {
           if (!content) return;
-          this.content = content;
+          this.componentContent = content;
         };
 
         return {
-          id,
-          isDark,
-          isHeadless,
-          isRight,
-          button,
-          content,
-          registerButton,
+          id: this.componentId,
+          component: this.component,
+          componentTrigger: this.componentTrigger,
+          componentContent: this.componentContent,
+          registerTrigger,
           registerContent,
         };
       },
@@ -50,22 +48,37 @@ export default {
       type: Boolean,
       default: false,
     },
-    dark: {
-      type: Boolean,
-      default: false,
-    },
-    right: {
-      type: Boolean,
-      default: false,
+    placement: {
+      type: String,
+      default: 'bottom-start',
     },
   },
 
   data() {
     return {
-      id: `popover-${genId()}`,
-      button: null,
-      content: null,
+      componentId: genId(),
+      /**
+       * Explaining the need for the floatingUiMinWidth data
+       *
+       * The floating ui is a result of two items:
+       *
+       * 1. Trigger: the action button
+       * 2. Content: the popper/wrapper that appears after triggering the action button
+       *
+       * By default, the content will match the triggers width.
+       * The problem with this is that the trigger width
+       * might be too small causing the content to not fit
+       * what is inside it properly. So, to avoid this,
+       * a min width is needed.
+       */
+      floatingUiMinWidth: 200,
     };
   },
+
+  computed: {
+    id() {
+      return `popover-${this.componentId}`;
+    },
+  },
 };
 </script>

package/src/components/Popover/VTPopoverContent.vue

@@ -1,82 +1,47 @@
 <template>
-  <transition
-    enter-active-class="duration-200 ease-out"
-    enter-class="translate-y-[15px] opacity-0"
-    enter-to-class="translate-y-0 opacity-100"
-    leave-active-class="duration-200 ease-in"
-    leave-class="translate-y-0 opacity-100"
-    leave-to-class="translate-y-[15px] opacity-0"
-    @after-leave="hide"
+  <FloatingUi
+    :visible="visible"
+    :id="id"
+    :headless="headless"
+    :class="{ 'popover-content': headless }"
+    :floating-ui-class="floatingUiClass"
   >
-    <div
-      v-show="visible"
-      :id="id"
-      :class="{
-        PopoverPanel: headless,
-        'absolute top-full mt-3 rounded-md shadow-300 ': !headless,
-        'bg-white': !dark,
-        'border-gray-700 bg-forest-default shadow-gray-700': dark,
-        'left-0': !right,
-        'right-0': right,
-      }"
-    >
-      <slot></slot>
-    </div>
-  </transition>
+    <slot></slot>
+  </FloatingUi>
 </template>
 
 <script>
-import { genId } from "../../utils/ids";
+import { floatingUiContentMixin } from '../../../mixins/floating-ui-content';
 
 export default {
-  name: "VTPopoverContent",
+  name: 'VTPopoverContent',
 
-  inject: ["api"],
+  mixins: [floatingUiContentMixin],
 
-  data() {
-    return {
-      id: `popover-panel-${genId()}`,
-      visible: false,
-    };
-  },
+  inject: ['apiPopover'],
 
   computed: {
-    dark() {
-      return this.api().isDark;
+    id() {
+      return `popover-content-${this.apiPopover().id}`;
     },
 
-    headless() {
-      return this.api().isHeadless;
+    component() {
+      return this.apiPopover().component;
     },
 
-    right() {
-      return this.api().isRight;
+    componentTrigger() {
+      return this.apiPopover().componentTrigger;
     },
   },
 
   mounted() {
-    this.api().registerContent(this);
-
-    // TODO: Create a directive or mixin for this
-    document.addEventListener("click", (e) => {
-      e.stopPropagation();
-      if (this.visible && !this.$el.contains(e.target)) this.hide();
-    });
-  },
-
-  destroyed() {
-    // TODO: Create a directive or mixin for this
-    document.removeEventListener("click", this.hide());
-  },
-
-  methods: {
-    show() {
-      this.visible = true;
-    },
+    const content = {
+      id: this.id,
+      show: this.show,
+      hide: this.hide,
+    };
 
-    hide() {
-      this.visible = false;
-    },
+    this.apiPopover().registerContent(content);
   },
 };
 </script>

package/src/components/Popover/VTPopoverDivider.vue

@@ -3,8 +3,6 @@
     :class="{
       PopoverDivider: headless,
       'h-[1px]': !headless,
-      'bg-white': !dark,
-      'bg-fd-500': dark,
     }"
   ></div>
 </template>
@@ -13,15 +11,10 @@
 export default {
   name: 'VTPopoverDivider',
 
-  inject: ['api'],
-
-  computed: {
-    dark() {
-      return this.api().isDark;
-    },
-
-    headless() {
-      return this.api().isHeadless;
+  props: {
+    headless: {
+      type: Boolean,
+      default: false,
     },
   },
 };

package/src/components/Popover/VTPopoverItem.vue

@@ -1,13 +1,12 @@
 <template>
   <component
     :is="as"
-    :class="{
-      PopoverItem: headless,
-      '-mx-3 flex min-w-max items-center gap-2 px-3 py-2 no-underline':
-        !headless,
-      'text-fd-500': !dark,
-      'text-white hover:bg-fd-450': dark,
-    }"
+    :class="[
+      // default styles
+      headless
+        ? 'popover-item'
+        : 'hover:bg-secondary-200/10 relative z-10 -mx-3 flex items-center gap-2 px-3 py-2 text-inherit no-underline',
+    ]"
     @click="onClick"
   >
     <slot></slot>
@@ -18,30 +17,38 @@
 export default {
   name: 'VTPopoverItem',
 
-  inject: ['api'],
+  inject: ['apiPopover'],
 
   props: {
-    to: {
-      type: [String, Object],
-      default: null,
+    headless: {
+      type: Boolean,
+      default: false,
     },
     href: {
       type: String,
       default: null,
     },
+    to: {
+      type: [String, Object],
+      default: null,
+    },
   },
 
   computed: {
     dark() {
-      return this.api().isDark;
+      return this.apiPopover().isDark;
     },
 
     headless() {
-      return this.api().isHeadless;
+      return this.apiPopover().isHeadless;
     },
 
     as() {
-      return this.href ? 'a' : this.to ? 'NuxtLink' : 'button';
+      return this.href
+        ? 'a'
+        : this.to
+        ? resolveComponent('NuxtLink')
+        : 'button';
     },
   },