@onereach/ui-components 5.0.2-beta.3154.0 → 5.0.2

package/src/components/or-select-v3/OrSelect.vue

@@ -39,14 +39,14 @@
         @blur="$emit('blur', $event)"
       >
         <template v-if="multiple">
-          <template v-if="!isEmptyModelValue">
+          <template v-if="proxyModelValue && proxyModelValue.length > 0">
             <slot
               name="valueTemplate"
               :model="proxyModelValue"
             >
               <OrTag
-                v-for="option in selection"
-                :key="option.value"
+                v-for="(option, i) in selection"
+                :key="i"
                 :variant="'reset'"
                 :disabled="disabled || readonly"
                 @reset="deselect(option)"
@@ -56,13 +56,8 @@
             </slot>
           </template>
 
-          <template v-else-if="popoverState === 'closed' || !enableSearch">
-            <span
-              :class="placeholderStyles"
-              :disabled="disabled ? '' : null"
-            >
-              {{ placeholder }}
-            </span>
+          <template v-else>
+            <span :class="placeholderStyles">{{ placeholder }}</span>
           </template>
         </template>
 
@@ -78,13 +73,8 @@
             </slot>
           </template>
 
-          <template v-else-if="popoverState === 'closed' || !enableSearch">
-            <span
-              :class="placeholderStyles"
-              :disabled="disabled ? '' : null"
-            >
-              {{ placeholder }}
-            </span>
+          <template v-else>
+            <span :class="placeholderStyles">{{ placeholder }}</span>
           </template>
         </template>
       </div>
@@ -93,30 +83,13 @@
         +{{ outerItems.length }}
       </template>
 
-      <template v-if="popoverState === 'open' && enableSearch && isDesktop">
-        <OrInput
-          ref="searchControl"
-          v-model="searchText"
-          :class="[...searchControlStyles, isEmptyModelValue && 'md:!ml-none']"
-          :type="'search'"
-          :placeholder="searchPlaceholder"
-          :size="size"
-          :disable-default-styles="true"
-          @click.stop
-        />
-      </template>
-
-      <template v-else>
-        <div :class="['grow']" />
-      </template>
-
       <div :class="['layout-inline-row', 'gap-sm']">
-        <template v-if="popoverState === 'closed' && !isEmptyModelValue">
+        <template v-if="!multiple && proxyModelValue">
           <OrIconButton
             :icon="'close'"
             :color="'inherit'"
             :disabled="readonly || disabled"
-            @click.stop="$emit('update:modelValue', multiple ? [] : undefined)"
+            @click.stop="$emit('update:modelValue', undefined)"
           />
         </template>
 
@@ -134,15 +107,14 @@
         :full-width="true"
         :custom-styles="dropdownStyles"
         :disable-default-styles="true"
-        @close="clearSearchControl()"
       >
-        <template v-if="popoverState === 'open' && enableSearch && isMobile">
+        <template v-if="enableSearch">
           <OrInput
-            ref="searchControl"
             v-model="searchText"
-            :class="searchControlStyles"
             :type="'search'"
-            :placeholder="searchPlaceholder"
+            :placeholder="'Search'"
+            :size="'s'"
+            :disable-default-styles="true"
           />
         </template>
 
@@ -157,7 +129,6 @@
                 <OrMenuItem
                   v-for="(option, i) in group"
                   :key="i"
-                  :selected="proxyModelValue && proxyModelValue.includes(option.value)"
                 >
                   <slot
                     name="optionTemplate"
@@ -168,7 +139,7 @@
                       v-model="proxyModelValue"
                       :value="option.value"
                       :label="option.label"
-                      :label-variant="'menu-item'"
+                      :label-variant="'input'"
                     />
                   </slot>
                 </OrMenuItem>
@@ -178,8 +149,6 @@
                 <OrMenuItem
                   v-for="(option, i) in group"
                   :key="i"
-                  :class="dropdownItemStyles"
-                  :selected="proxyModelValue === option.value"
                   @click.stop="proxyModelValue = option.value; close()"
                 >
                   <slot
@@ -199,13 +168,12 @@
               <OrMenuItem
                 v-for="(option, i) in resolvedSearchOptions"
                 :key="i"
-                :selected="proxyModelValue && proxyModelValue.includes(option.value)"
               >
                 <OrCheckbox
                   v-model="proxyModelValue"
                   :value="option.value"
                   :label="option.label"
-                  :label-variant="'menu-item'"
+                  :label-variant="'input'"
                 >
                   <slot
                     name="optionTemplate"
@@ -220,8 +188,6 @@
               <OrMenuItem
                 v-for="(option, i) in resolvedSearchOptions"
                 :key="i"
-                :class="dropdownItemStyles"
-                :selected="proxyModelValue === option.value"
                 @click.stop="proxyModelValue = option.value; close()"
               >
                 <slot
@@ -271,7 +237,7 @@
 <script lang="ts">
 import { computed, defineComponent, getCurrentInstance, onMounted, PropType, ref, toRef, watch } from 'vue-demi';
 import { DropdownClose, DropdownOpen } from '../../directives';
-import { useOverflow, useProxyModelValue, useResponsive } from '../../hooks';
+import { useOverflow, useProxyModelValue } from '../../hooks';
 import { isEmptyValue } from '../../utils/isEmptyValue';
 import { OrCheckboxV3 as OrCheckbox } from '../or-checkbox-v3';
 import { OrErrorV3 as OrError } from '../or-error-v3';
@@ -286,10 +252,12 @@ import { OrMenuItemV3 as OrMenuItem } from '../or-menu-item-v3';
 import { OrPopoverV3 as OrPopover } from '../or-popover-v3';
 import { OrTagV3 as OrTag } from '../or-tag-v3';
 import { SelectSize } from './props';
-import { Select, SelectControl, SelectDropdown, SelectDropdownDefault, SelectDropdownFlipped, SelectDropdownItem, SelectNoSearchResults, SelectPlaceholder, SelectSearchControl } from './styles';
+import { SelectControl, SelectDropdown, SelectDropdownDefault, SelectDropdownFlipped, SelectNoSearchResults, SelectPlaceholder, SelectRoot } from './styles';
 import { SelectGroupByFunction, SelectOption, SelectSearchFunction } from './types';
 
 export default defineComponent({
+  name: 'OrSelect',
+
   components: {
     OrCheckbox,
     OrError,
@@ -351,11 +319,6 @@ export default defineComponent({
       default: undefined,
     },
 
-    searchPlaceholder: {
-      type: String,
-      default: undefined,
-    },
-
     hint: {
       type: String,
       default: undefined,
@@ -429,8 +392,6 @@ export default defineComponent({
 
     const control = ref<HTMLElement>();
 
-    const searchControl = ref<HTMLElement>();
-
     const popover = ref<InstanceType<typeof OrPopover>>();
     const popoverRoot = computed(() => popover.value?.root);
     const popoverState = computed(() => popover.value?.state);
@@ -438,17 +399,13 @@ export default defineComponent({
     // Styles
     const rootStyles = computed(() => [
       'or-select-v3',
-      ...Select,
+      ...SelectRoot,
     ]);
 
     const controlStyles = computed(() => [
       ...SelectControl,
     ]);
 
-    const searchControlStyles = computed(() => [
-      ...SelectSearchControl,
-    ]);
-
     const placeholderStyles = computed(() => [
       ...SelectPlaceholder,
     ]);
@@ -458,10 +415,6 @@ export default defineComponent({
       ...popover.value?.isFlipped ? SelectDropdownFlipped : SelectDropdownDefault,
     ]);
 
-    const dropdownItemStyles = computed(() => [
-      ...SelectDropdownItem,
-    ]);
-
     const noSearchResultsStyles = computed(() => [
       ...SelectNoSearchResults,
     ]);
@@ -489,9 +442,7 @@ export default defineComponent({
       const modelValue = proxyModelValue.value;
 
       if (Array.isArray(modelValue)) {
-        return modelValue
-          .map((value) => props.options.find((option) => option.value === value))
-          .filter((value) => !!value);
+        return modelValue.map((value) => props.options.find((option) => option.value === value));
       }
 
       return props.options.find((option) => option.value === modelValue);
@@ -570,37 +521,20 @@ export default defineComponent({
       }
     }
 
-    function clearSearchControl(): void {
-      searchText.value = '';
-    }
-
     const { innerItems, outerItems } = useOverflow(control);
 
-    // Utils
-    const { isDesktop, isMobile } = useResponsive();
-
-    // Effects
-    watch(searchControl, (control) => {
-      setTimeout(() => {
-        control?.focus();
-      });
-    });
-
     return {
       root,
       inputBox,
       inputBoxRoot,
       control,
-      searchControl,
       popover,
       popoverRoot,
       popoverState,
       rootStyles,
       controlStyles,
-      searchControlStyles,
       placeholderStyles,
       dropdownStyles,
-      dropdownItemStyles,
       noSearchResultsStyles,
       controlId,
       proxyModelValue,
@@ -614,11 +548,8 @@ export default defineComponent({
       toggle,
       close,
       deselect,
-      clearSearchControl,
       innerItems,
       outerItems,
-      isDesktop,
-      isMobile,
     };
   },
 });

package/src/components/or-select-v3/styles.ts

@@ -1,4 +1,4 @@
-export const Select: string[] = [
+export const SelectRoot: string[] = [
   // Layout
   'layout-column',
 
@@ -10,8 +10,8 @@ export const SelectControl: string[] = [
   // Layout
   'layout-row',
 
-  // Layout (empty)
-  'empty:hidden',
+  // Box
+  'w-full',
 
   // Overflow
   'overflow-hidden',
@@ -23,15 +23,6 @@ export const SelectControl: string[] = [
   'typography-inherit',
 ];
 
-export const SelectSearchControl: string[] = [
-  // Layout
-  'grow',
-
-  // Spacing
-  '!mx-lg md:!-mx-sm+',
-  '!my-sm md:!my-none',
-];
-
 export const SelectPlaceholder: string[] = [
   // Theme
   'theme-foreground-outline',
@@ -75,14 +66,6 @@ export const SelectDropdownFlipped: string[] = [
   '!border-b-0',
 ];
 
-export const SelectDropdownItem: string[] = [
-  // Typography
-  '!typography-body-2-regular',
-
-  // Typography (selected)
-  'selected:!typography-body-2-semibold',
-];
-
 export const SelectNoSearchResults: string[] = [
   // Layout
   'layout-row justify-center',

package/src/directives/dropdown-close.ts

@@ -2,6 +2,9 @@ import { DirectiveBinding, DirectiveHook, isVue3 } from 'vue-demi';
 
 const created: DirectiveHook = (element: HTMLElement, binding: DirectiveBinding<(event: Event) => void>) => {
   element.addEventListener('keydown', (event) => {
+    event.preventDefault();
+    event.stopPropagation();
+
     if (['Escape'].includes(event.key)) {
       binding.value(event);
     }

package/src/directives/dropdown-open.ts

@@ -2,14 +2,17 @@ import { DirectiveBinding, DirectiveHook, isVue3 } from 'vue-demi';
 
 const created: DirectiveHook = (element: HTMLElement, binding: DirectiveBinding<(event: Event) => void>) => {
   element.addEventListener('keydown', (event) => {
+    event.preventDefault();
+    event.stopPropagation();
+
     if (['Enter', ' '].includes(event.key)) {
       binding.value(event);
     }
   });
 
   element.addEventListener('click', (event) => {
-    event.stopPropagation();
     event.preventDefault();
+    event.stopPropagation();
 
     binding.value(event);
   });