@veritree/ui 0.19.2-20 → 0.19.2-22

package/src/components/Listbox/VTListboxItem.vue

@@ -5,30 +5,47 @@
     :aria-disabled="disabled"
     :aria-selected="String(selected)"
     :tabindex="tabIndex"
-    role="option"
+    :role="'option'"
     @click.stop="onClick"
-    @keydown.down.prevent="focusPreviousItem"
-    @keydown.up.prevent="focusNextItem"
+    @keydown.prevent="onKeydown"
+    @keydown.down.prevent="focusNextItem"
+    @keydown.up.prevent="focusPreviousItem"
     @keydown.home.prevent="focusFirstItem"
     @keydown.end.prevent="focusLastItem"
     @keydown.esc.stop="onKeyEsc"
     @keydown.enter.prevent="onClick"
-    @mousemove="onMousemove"
-    @mouseout="onMouseleave"
     @keydown.tab.prevent
   >
+    <span v-if="apiListbox().multiple">
+      <input
+        v-model="apiListbox().$mutable.valueComputed"
+        :id="`${id}-checkbox`"
+        :value="value"
+        type="checkbox"
+        @click.stop
+        @change="onChange"
+      />
+    </span>
     <slot></slot>
+    <span v-if="wasSelected" class="ml-auto">
+      <IconCheckOutline class="text-secondary-200 h-5 w-5" />
+    </span>
   </li>
 </template>
 
 <script>
+import { IconCheckOutline } from '@veritree/icons';
 import { floatingUiItemMixin } from '../../../mixins/floating-ui-item';
 
+let keydownSearchTimeout = null;
+
 export default {
   name: 'VTListboxItem',
 
   mixins: [floatingUiItemMixin],
 
+  components: { IconCheckOutline },
+
   inject: ['apiListbox'],
 
   props: {
@@ -40,14 +57,118 @@ export default {
 
   data() {
     return {
+      // apiInjected is used in the mixin floating-ui-item
       apiInjected: this.apiListbox,
       componentName: 'listbox-item',
     };
   },
 
   computed: {
+    componentContent() {
+      return this.apiInjected().componentContent;
+    },
+
+    hasComponentSearch() {
+      return this.apiListbox().componentSearch !== null;
+    },
+
     search() {
-      return this.apiInjected().search;
+      return this.apiListbox().search;
+    },
+
+    /**
+     * The difference between selected (in the mixin) and
+     * wasSelected is that selected is more related to
+     * the aria-selected while wasSelected is the
+     * real selected option for the listbox
+     */
+    wasSelected() {
+      return (
+        JSON.stringify(this.value) ===
+        JSON.stringify(this.apiListbox().$mutable.valueComputed)
+      );
+    },
+  },
+
+  mounted() {
+    if (this.wasSelected) {
+      /**
+       * There are some conflicts between the portal and the interaction
+       * with the element. It's unclear why it happens but if no delay,
+       * the floating ui content, that is placed in the body by the
+       * vue portal plugin, will not appear close to the trigger.
+       *
+       * Kind hacky but no time do find real solution.
+       */
+      setTimeout(() => {
+        this.hasComponentSearch ? this.select() : this.focus();
+      }, 1);
+    }
+  },
+
+  methods: {
+    /**
+     * This event is mainly for the search since other keydown
+     * events are handled with vue v-on (@) directive
+     *
+     * @param {Object} event
+     */
+    onKeydown(e) {
+      const isLetter = e.code.includes('Key');
+      const isNumber = e.code.includes('Numpad') || e.code.includes('Digit');
+      const isBackspace = e.code === 'Backspace';
+      const isSpace = e.code === 'Space';
+
+      if (isLetter || isNumber || isBackspace || isSpace) {
+        this.doSearch(e);
+      }
+    },
+
+    /**
+     * Searches and focus the options that matches the
+     * value typed when the item is selected.
+     *
+     * @param {Object} event
+     */
+    doSearch(e) {
+      clearTimeout(keydownSearchTimeout);
+
+      /**
+       * Add letter or number to search data in component
+       * root, so it is available for all children
+       */
+      this.apiListbox().pushSearch(e.key);
+
+      /**
+       * Let's get the item selected index at the moment the search
+       * starts to later unselect it if necessary
+       */
+      const itemSelectedIndex = this.getItemSelectedIndex();
+
+      /**
+       * As we have now the search data with value, let's go
+       * through all items and check if the search matches
+       * with any of the items to get its index value
+       */
+      const newItemSelectedIndex = this.items.findIndex((item) =>
+        item.text.toLowerCase().includes(this.search.toLowerCase())
+      );
+
+      /**
+       * Focus the item that matches the search, and if no match
+       * we will just remove the selected state from item
+       */
+      newItemSelectedIndex >= 0
+        ? this.setFocusToItem(itemSelectedIndex, newItemSelectedIndex)
+        : this.unselect();
+
+      /**
+       * Clear the search data value after some time
+       * to allow other searches to be performed
+       */
+      keydownSearchTimeout = setTimeout(() => {
+        this.apiListbox().clearSearch();
+      }, 1000);
     },
   },
 };

package/src/components/Listbox/VTListboxLabel.vue

@@ -1,10 +1,9 @@
 <template>
   <component
     :is="as"
-    :class="{
-      ListboxLabel: headless,
-      'mb-2 block text-xs font-normal uppercase': !headless,
-    }"
+    :class="[
+      headless ? 'listbox-label' : 'mb-2 block text-xs font-normal uppercase',
+    ]"
   >
     <slot></slot>
   </component>

package/src/components/Listbox/VTListboxList.vue

@@ -2,7 +2,9 @@
   <ul
     :id="id"
     :class="[
-      headless ? 'listbox-list' : 'max-h-[160px] w-auto overflow-y-auto -mx-3',
+      headless
+        ? 'listbox-list'
+        : '-mx-3 max-h-[160px] w-auto overflow-y-auto scroll-auto',
     ]"
   >
     <slot></slot>
@@ -27,28 +29,5 @@ export default {
       return `listbox-list-${this.apiListbox().id}`;
     },
   },
-
-  // mounted() {
-  //   const list = {
-  //     el: this.$el,
-  //   };
-
-  //   this.apiListbox().registerList(list);
-  // },
-
-  methods: {
-    // Mousemove instead of mouseover to support keyboard navigation.
-    // The problem with mouseover is that when scrolling (scrollIntoView),
-    // mouseover event gets triggered.
-    // setMousemove() {
-    //   this.isMousemove = true;
-    // },
-    // unsetMousemove() {
-    //   this.isMousemove = false;
-    // },
-    // getMousemove() {
-    //   return this.isMousemove;
-    // },
-  },
 };
 </script>

package/src/components/Listbox/VTListboxSearch.vue

@@ -1,18 +1,22 @@
 <template>
-  <input
-    :value="modelValue"
-    type="text"
-    :class="classComputed"
-    @input="onInput"
-    @click.stop
-    @keydown.down.prevent="focusNextItem"
-    @keydown.up.prevent="focusPreviousItem"
-    @keydown.home.prevent="focusFirstItem"
-    @keydown.end.prevent="focusLastItem"
-    @keydown.enter.prevent="onKeyEnter"
-    @keydown.esc.stop="hide"
-    @keydown.tab.prevent="hide"
-  />
+  <div class="-mx-3 -mt-2">
+    <input
+      ref="search"
+      :value="modelValue"
+      v-bind="$attrs"
+      type="text"
+      class="leading-0 font-inherit w-full max-w-full appearance-none border-b border-solid border-gray-300 py-2 px-3 text-base text-inherit"
+      @input="onInput"
+      @click.stop
+      @keydown.down.prevent="focusNextItem"
+      @keydown.up.prevent="focusPreviousItem"
+      @keydown.home.prevent="focusFirstItem"
+      @keydown.end.prevent="focusLastItem"
+      @keydown.enter.prevent="onKeyEnter"
+      @keydown.esc.stop="hide"
+      @keydown.tab.prevent="hide"
+    />
+  </div>
 </template>
 
 <script>
@@ -23,12 +27,14 @@ export default {
 
   mixins: [formControlMixin],
 
+  inheritAttrs: false,
+
   inject: ['apiListbox'],
 
   data() {
     return {
       name: 'listbox-search',
-      index: -1,
+      index: null,
       search: '',
     };
   },
@@ -38,10 +44,6 @@ export default {
       return this.apiListbox().componentTrigger;
     },
 
-    componentContent() {
-      return this.apiListbox().componentContent;
-    },
-
     items() {
       return this.apiListbox().items;
     },
@@ -52,12 +54,12 @@ export default {
   },
 
   mounted() {
-    const search = {
+    const componentSearch = {
       el: this.$el,
     };
 
-    this.apiListbox().registerSearch(search);
-    this.$nextTick(() => setTimeout(() => this.$el.focus(), 150));
+    this.apiListbox().registerSearch(componentSearch);
+    this.$nextTick(() => setTimeout(() => this.$refs.search.focus(), 150));
   },
 
   beforeUnmount() {
@@ -67,56 +69,61 @@ export default {
 
   methods: {
     focusNextItem() {
-      if (this.index !== -1) {
-        this.unselectItem();
-      }
-
-      this.index++;
+      const selectedIndex = this.getItemSelectedIndex();
+      const isLast = selectedIndex === this.items.length - 1;
+      const firstItemIndex = 0;
+      const nextItemIndex = selectedIndex + 1;
+      const newSelectedIndex = isLast ? firstItemIndex : nextItemIndex;
 
-      if (this.index > this.items.length - 1) {
-        this.index = 0;
-      }
-
-      if (this.item) {
-        this.item.select();
-      }
+      this.selectItem(selectedIndex, newSelectedIndex);
     },
 
     focusPreviousItem() {
-      this.unselectItem();
-
-      this.index--;
-
-      if (this.index < 0) {
-        this.index = this.items.length - 1;
-      }
+      const selectedIndex = this.getItemSelectedIndex();
+      const isFirst = selectedIndex === 0;
+      const lastItemIndex = this.items.length - 1;
+      const previousItemIndex = selectedIndex - 1;
+      const newSelectedIndex = isFirst ? lastItemIndex : previousItemIndex;
 
-      this.item.select();
+      this.selectItem(selectedIndex, newSelectedIndex);
     },
 
     focusFirstItem() {
-      this.unselectItem();
-      this.index = 0;
-      this.item.select();
+      const selectedIndex = this.getItemSelectedIndex();
+      const newSelectedIndex = 0;
+
+      this.selectItem(selectedIndex, newSelectedIndex);
     },
 
     focusLastItem() {
-      this.unselectItem();
-      this.index = this.items.length - 1;
-      this.item.select();
+      const selectedIndex = this.getItemSelectedIndex();
+      const newSelectedIndex = this.items.length - 1;
+
+      this.selectItem(selectedIndex, newSelectedIndex);
     },
 
-    unselectItem() {
-      const isMousemove = this.componentContent.getMousemove();
+    selectItem(selectedIndex, newSelectedIndex) {
+      // before selecting, let's unselect selected
+      // item that were previously focused
+      if (selectedIndex >= 0) {
+        this.items[selectedIndex].unselect();
+      }
 
-      if (isMousemove) {
-        this.componentContent.unsetMousemove();
-        this.items.forEach((item) => item.unselect());
+      // select new item
+      if (this.items[newSelectedIndex]) {
+        this.index = newSelectedIndex;
+        this.items[newSelectedIndex].select();
       }
+    },
 
+    unselectItem() {
       if (this.item) this.item.unselect();
     },
 
+    getItemSelectedIndex() {
+      return this.items.findIndex((item) => item.isSelected());
+    },
+
     onInput(event) {
       this.index = 0;
       if (this.item) this.item.select();

package/src/components/Listbox/VTListboxTrigger.vue

@@ -24,7 +24,10 @@
 </template>
 
 <script>
-import { formControlMixin } from '../../../mixins/form-control';
+import {
+  formControlMixin,
+  formControlStyleMixin,
+} from '../../../mixins/form-control';
 import { IconChevronDown } from '@veritree/icons';
 
 export default {
@@ -32,20 +35,23 @@ export default {
 
   components: { IconChevronDown },
 
-  mixins: [formControlMixin],
+  mixins: [formControlMixin, formControlStyleMixin],
 
   inject: ['apiListbox'],
 
   data() {
     return {
-      name: 'listbox-button',
+      name: 'listbox-trigger',
       expanded: false,
       hasPopup: false,
-      id: null,
     };
   },
 
   computed: {
+    id() {
+      return `listbox-trigger-${this.apiListbox().id}`;
+    },
+
     componentContent() {
       return this.apiListbox().componentContent;
     },
@@ -64,8 +70,6 @@ export default {
   },
 
   mounted() {
-    this.id = `listbox-trigger-${this.apiListbox().id}`;
-
     const trigger = {
       el: this.$el,
       cancel: this.cancel,
@@ -100,6 +104,7 @@ export default {
       if (!this.componentContent) {
         return;
       }
+
       this.expanded = false;
 
       this.hideComponentContent();
@@ -119,6 +124,7 @@ export default {
 
     onClick(e) {
       this.init(e);
+      this.$emit('click');
     },
 
     onKeyDownOrUp(e) {

package/src/components/Popover/VTPopover.vue

@@ -48,11 +48,30 @@ export default {
       type: Boolean,
       default: false,
     },
+    placement: {
+      type: String,
+      default: 'bottom-start',
+    },
   },
 
   data() {
     return {
       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,
     };
   },
 

package/src/components/ProgressBar/VTProgressBar.vue

@@ -7,7 +7,7 @@
     ]"
     role="progressbar"
     aria-valuemin="0"
-    aria-valuemax="100"
+    :aria-valuemax="max"
     :aria-valuenow="value"
     :aria-label="label"
   >
@@ -15,9 +15,9 @@
       :class="[
         headless
           ? 'progress-bar__indicator'
-          : 'absolute left-0 h-full bg-secondary-300 transition-all',
+          : 'bg-secondary-200 absolute left-0 h-full transition-all',
       ]"
-      :style="{ width: `${value}%` }"
+      :style="{ width: `${percentageComputed}%` }"
     ></div>
   </div>
 </template>
@@ -37,6 +37,24 @@ export default {
       type: [String, Number],
       default: 0,
     },
+    max: {
+      type: [String, Number],
+      default: 100,
+    },
+    percentage: {
+      type: [String, Number],
+      default: null,
+    },
+  },
+
+  computed: {
+    percentageComputed() {
+      if (typeof this.percentage !== 'undefined' && this.percentage !== null) {
+        return this.percentage;
+      }
+
+      return (this.value / this.max) * 100;
+    },
   },
 };
 </script>

package/src/components/Skeleton/VTSkeleton.vue

@@ -0,0 +1,11 @@
+<template>
+  <div class="animate-pulse">
+    <slot />
+  </div>
+</template>
+
+<script>
+export default {
+  name: 'VTSkeleton',
+};
+</script>

package/src/components/Skeleton/VTSkeletonItem.vue

@@ -0,0 +1,9 @@
+<template>
+  <div class="bg-gray-300" />
+</template>
+
+<script>
+export default {
+  name: 'VTSkeletonItem',
+};
+</script>

package/src/components/Tabs/VTTab.vue

@@ -18,11 +18,11 @@
     ]"
     role="tab"
     type="button"
-    @click.stop="onClick"
+    @click.prevent.stop="onClick"
     @keydown="onKeyDown"
     @blur="onBlur"
   >
-    <slot></slot>
+    <slot :selected="selected"></slot>
   </button>
 </template>
 
@@ -32,7 +32,7 @@ import { keys } from '../../utils/keyboard';
 export default {
   name: 'VTTabItem',
 
-  inject: ['apiTabs'],
+  inject: ['api'],
 
   props: {
     headless: {
@@ -43,6 +43,7 @@ export default {
 
   data() {
     return {
+      api: this.api(),
       index: null,
       indexFocus: null,
       selected: false,
@@ -56,11 +57,11 @@ export default {
   },
 
   mounted() {
-    this.apiTabs().registerTab(this);
+    this.api.registerTab(this);
   },
 
   beforeUnmount() {
-    this.apiTabs().unregisterTab(this.index);
+    this.api.unregisterTab(this.index);
   },
 
   methods: {
@@ -77,11 +78,11 @@ export default {
     },
 
     focusFirstTab() {
-      this.apiTabs().focusTab(0);
+      this.api.focusTab(0);
     },
 
     focusLastTab(lastTabIndex) {
-      this.apiTabs().focusTab(lastTabIndex);
+      this.api.focusTab(lastTabIndex);
     },
 
     focusPreviousTab(lastTabIndex) {
@@ -90,7 +91,7 @@ export default {
         return;
       }
 
-      this.apiTabs().focusTab(this.indexFocus - 1);
+      this.api.focusTab(this.indexFocus - 1);
     },
 
     focusNextTab(lastTabIndex) {
@@ -99,17 +100,18 @@ export default {
         return;
       }
 
-      this.apiTabs().focusTab(this.indexFocus + 1);
+      this.api.focusTab(this.indexFocus + 1);
     },
 
     onClick() {
-      this.apiTabs().selectTab(this.index);
+      this.api.selectTab(this.index);
       this.$emit('change');
+      this.$emit('click');
     },
 
     onKeyDown(event) {
       const key = event.key;
-      const lastTabIndex = this.apiTabs().getTabsLength() - 1;
+      const lastTabIndex = this.api.getTabsLength() - 1;
 
       if (this.indexFocus === null) {
         this.indexFocus = this.index;