From 21329e8239b909bd55719793cf2802d9ed9953ed Mon Sep 17 00:00:00 2001
From: jd-solanki <jdsolanki0001@gmail.com>
Date: Thu, 9 Feb 2023 17:12:41 +0530
Subject: [PATCH] feat(select)!: reuse `AFloating` & `AList`components

---
 .../demos/list/DemoListVModelSupport.vue      |  12 +-
 .../demos/select/DemoSelectBasic.vue          |  10 +-
 .../src/components/floating/AFloating.vue     |   7 ++
 .../src/components/floating/middlewares.ts    |  18 ++-
 .../anu-vue/src/components/list/AList.vue     |  52 +++++---
 .../anu-vue/src/components/select/ASelect.vue | 113 ++++++------------
 .../preset-theme-default/src/scss/index.scss  |  12 +-
 .../preset-theme-default/src/shortcuts.ts     |   5 +-
 8 files changed, 109 insertions(+), 120 deletions(-)

diff --git a/docs/components/demos/list/DemoListVModelSupport.vue b/docs/components/demos/list/DemoListVModelSupport.vue
index d5cd504a..d12a9b81 100644
--- a/docs/components/demos/list/DemoListVModelSupport.vue
+++ b/docs/components/demos/list/DemoListVModelSupport.vue
@@ -8,8 +8,8 @@ const items = [
   { text: 'Cake gummi', disabled: true },
 ]
 
-const itemsPropSelection = ref(0)
-const slotSelection = ref(0)
+const itemsPropSelection = ref<typeof items[number] | null>(null)
+const slotSelection = ref<typeof items[number] | null>(null)
 </script>
 
 <template>
@@ -25,7 +25,7 @@ const slotSelection = ref(0)
           <hr class="my-2">
           <AList
             class="mb-0"
-            :items="[{ subtitle: `Selected: ${itemsPropSelection}` }]"
+            :items="[{ subtitle: `Selected: ${itemsPropSelection && itemsPropSelection.text}` }]"
           />
         </template>
       </AList>
@@ -44,8 +44,8 @@ const slotSelection = ref(0)
             :key="item.text"
             :text="item.text"
             :value="index"
-            :disable="item.disable"
-            :is-active="slotSelection === index"
+            :disabled="item.disabled"
+            :is-active="slotSelection?.text === item.text"
             @click="handleListItemClick(item, index)"
           />
         </template>
@@ -53,7 +53,7 @@ const slotSelection = ref(0)
           <hr class="my-2">
           <AList
             class="mb-0"
-            :items="[{ subtitle: `Selected: ${slotSelection}` }]"
+            :items="[{ subtitle: `Selected: ${slotSelection && slotSelection.text}` }]"
           />
         </template>
       </AList>
diff --git a/docs/components/demos/select/DemoSelectBasic.vue b/docs/components/demos/select/DemoSelectBasic.vue
index 435ddc59..da3ac6de 100644
--- a/docs/components/demos/select/DemoSelectBasic.vue
+++ b/docs/components/demos/select/DemoSelectBasic.vue
@@ -8,11 +8,11 @@ const fruits = ['banana', 'apple', 'watermelon', 'orange']
 // Framework
 const selectedFramework = ref()
 const frameworks = [
-  { label: 'VueJS', value: 'vue' },
-  { label: 'ReactJS', value: 'react' },
-  { label: 'AngularJS', value: 'angular' },
-  { label: 'SolidJS', value: 'solid' },
-  { label: 'AlpineJS', value: 'alpine' },
+  { text: 'VueJS', value: 'vue' },
+  { text: 'ReactJS', value: 'react' },
+  { text: 'AngularJS', value: 'angular' },
+  { text: 'SolidJS', value: 'solid' },
+  { text: 'AlpineJS', value: 'alpine' },
 ]
 </script>
 
diff --git a/packages/anu-vue/src/components/floating/AFloating.vue b/packages/anu-vue/src/components/floating/AFloating.vue
index b9aa1d1f..4bdfd2be 100644
--- a/packages/anu-vue/src/components/floating/AFloating.vue
+++ b/packages/anu-vue/src/components/floating/AFloating.vue
@@ -16,6 +16,7 @@ const emit = defineEmits<{
 
 defineOptions({
   name: 'AFloating',
+  inheritAttrs: false,
 })
 
 const { teleportTarget } = useTeleport()
@@ -106,6 +107,11 @@ if (props.modelValue === undefined) {
     }
   }
 }
+
+// Expose: https://vuejs.org/api/sfc-script-setup.html#defineexpose
+defineExpose({
+  refFloating,
+})
 </script>
 
 <template>
@@ -117,6 +123,7 @@ if (props.modelValue === undefined) {
     <Transition :name="props.transition || undefined">
       <div
         v-show="props.modelValue ?? isFloatingElVisible"
+        v-bind="$attrs"
         ref="refFloating"
         class="a-floating"
         :style="{
diff --git a/packages/anu-vue/src/components/floating/middlewares.ts b/packages/anu-vue/src/components/floating/middlewares.ts
index 0e0c50ad..058718b1 100644
--- a/packages/anu-vue/src/components/floating/middlewares.ts
+++ b/packages/anu-vue/src/components/floating/middlewares.ts
@@ -1,14 +1,12 @@
 import type { ElementRects } from '@floating-ui/vue'
 import type { Ref } from 'vue'
 
-export const sameWidth = (floatingEl: Ref<HTMLElement>) => {
-  return {
-    name: 'sameWidth',
-    fn: ({ rects, x, y }: { rects: ElementRects; x: number; y: number }) => {
-      // Set width of reference to floating
-      floatingEl.value.style.minWidth = `${rects.reference.width}px`
+export const sameWidth = (floatingEl: Ref<HTMLElement>) => ({
+  name: 'sameWidth',
+  fn: ({ rects, x, y }: { rects: ElementRects; x: number; y: number }) => {
+    // Set width of reference to floating
+    unrefElement(floatingEl).style.minWidth = `${rects.reference.width}px`
 
-      return { x, y }
-    },
-  }
-}
+    return { x, y }
+  },
+})
diff --git a/packages/anu-vue/src/components/list/AList.vue b/packages/anu-vue/src/components/list/AList.vue
index b9ed608e..a40f8d8d 100644
--- a/packages/anu-vue/src/components/list/AList.vue
+++ b/packages/anu-vue/src/components/list/AList.vue
@@ -3,62 +3,83 @@ import type { ExtractPropTypes, PropType } from 'vue'
 import type { ListItemProps } from '@/components/list-item'
 import { AListItem } from '@/components/list-item'
 import { useGroupModel } from '@/composables'
-import { isEmptyArray } from '@/utils/helpers'
+import { spacing as spacingProp } from '@/composables/useProps'
+import { useSpacing } from '@/composables/useSpacing'
+
+type ListItem = ListItemProps | string
 
 const props = defineProps({
   /**
    * Items to render in list
    */
-  items: {
-    type: Array as PropType<ListItemProps[]>,
+  'items': {
+    type: Array as PropType<ListItem[]>,
     default: () => [],
   },
 
   /**
    * Enable selecting multiple list items
    */
-  multi: Boolean,
+  'multi': Boolean,
 
   /**
    * Bind v-model value to selected list item
    */
-  modelValue: null,
+  'modelValue': null,
 
   /**
    * By default when icon props are used icon rendered at start. Use `iconAppend` to render icon at end.
    */
-  iconAppend: Boolean,
+  'iconAppend': Boolean,
 
   /**
    * By default when avatar props are used avatar is added at start. Use `avatarAppend` to render avatar at end.
    */
-  avatarAppend: Boolean,
+  'avatarAppend': Boolean,
+
+  // ℹ️ Workaround for checking if event is present on component instance: https://github.com/vuejs/core/issues/5220#issuecomment-1007488240
+  'onClick:item': Function,
+
+  'spacing': spacingProp,
 })
 
 const emit = defineEmits<{
   (e: 'update:modelValue', value: (ExtractPropTypes<typeof props>)['modelValue']): void
+
+  // ℹ️ Fix type => (e: 'click:item', value: (ExtractPropTypes<typeof props>)['items'][number]): void
+  (e: 'click:item', value: { index: number; item: ListItem; value: any }): void
 }>()
 
 defineOptions({
   name: 'AList',
 })
 
+const spacing = useSpacing(toRef(props, 'spacing'))
+
+const extractItemValueFromItemOption = (item: ListItem) => typeof item === 'string' ? item : (item.value || item)
+
 const { options, select: selectListItem, value } = useGroupModel({
-  options: !isEmptyArray(props.items) && props.items[0].value
-    ? props.items.map(i => i.value)
-    : props.items.length,
+  options: props.items.map(i => extractItemValueFromItemOption(i)),
   multi: props.multi,
 })
 
 // const isActive = computed(() => options.value[itemIndex].isSelected)
-const handleListItemClick = (item: ListItemProps, index: number) => {
-  selectListItem(item.value || index)
+const handleListItemClick = (item: ListItem, index: number) => {
+  selectListItem(extractItemValueFromItemOption(item) || index)
   emit('update:modelValue', value.value)
+  emit('click:item', {
+    index,
+    item,
+    value: value.value,
+  })
 }
 </script>
 
 <template>
-  <ul class="a-list grid gap-$a-list-gap">
+  <ul
+    class="a-list grid gap-$a-list-gap"
+    :style="[{ '--a-spacing': spacing / 100 }]"
+  >
     <!-- 👉 Slot: before -->
     <li v-if="$slots.before">
       <slot name="before" />
@@ -69,13 +90,14 @@ const handleListItemClick = (item: ListItemProps, index: number) => {
       <AListItem
         v-for="(item, index) in props.items"
         :key="index"
-        v-bind="item"
+        :text="typeof item === 'string' ? item : undefined"
+        v-bind="typeof item === 'string' ? {} : item"
         :avatar-append="props.avatarAppend"
         :icon-append="props.iconAppend"
         :is-active="options[index].isSelected as unknown as boolean"
         :value="props.modelValue !== undefined ? options[index] : undefined"
         v-on="{
-          click: item.value || (props.modelValue !== undefined)
+          click: props['onClick:item'] || (props.modelValue !== undefined)
             ? () => handleListItemClick(item, index)
             : null,
         }"
diff --git a/packages/anu-vue/src/components/select/ASelect.vue b/packages/anu-vue/src/components/select/ASelect.vue
index 2e34f17a..288197a6 100644
--- a/packages/anu-vue/src/components/select/ASelect.vue
+++ b/packages/anu-vue/src/components/select/ASelect.vue
@@ -1,26 +1,27 @@
 <script lang="ts" setup>
-import { autoUpdate, computePosition, flip, offset, shift } from '@floating-ui/dom'
+import { flip, offset, shift } from '@floating-ui/vue'
 import { defu } from 'defu'
 import type { ExtractPropTypes, PropType } from 'vue'
+import { ACard, AList } from '@/components'
 import { ABaseInput, baseInputProps } from '@/components/base-input'
-
-import { useTeleport } from '@/composables/useTeleport'
+import { AFloating, sameWidthFloatingUIMiddleware } from '@/components/floating'
+import type { ListItemProps } from '@/components/list-item'
 import { isObject } from '@/utils/helpers'
 
 export interface ObjectOption { label: string; value: string | number }
-type SelectOption = string | number | ObjectOption
 
 const props = defineProps(defu({
   // ℹ️ If we want any type need to set `propName: { type: null }`. Using `propName: null` will omit (disable) the prop.
   modelValue: { type: null },
   options: {
-    type: [String, Number, Object] as PropType<SelectOption[]>,
+    type: Array as PropType<ListItemProps[]>,
     default: () => [],
   },
   emitObject: Boolean,
 
   // ℹ️ If we want any type need to set `propName: { type: null }`. Using `propName: null` will omit (disable) the prop.
   optionsWrapperClasses: { type: null },
+  listClasses: { type: null },
 }, baseInputProps))
 
 const emit = defineEmits<{
@@ -36,52 +37,14 @@ defineOptions({
 
 const _baseInputProps = reactivePick(props, Object.keys(baseInputProps) as Array<keyof typeof baseInputProps>)
 
-const { teleportTarget } = useTeleport()
-const isMounted = useMounted()
-
 // SECTION Floating
 // Template refs
 const refReference = ref()
-const selectRef = ref<HTMLSelectElement>()
 const refFloating = ref()
+const selectRef = ref<HTMLSelectElement>()
 
-const isObjectOption = (option: SelectOption) => isObject(option) && 'label' in option && 'value' in option
 const isOptionsVisible = ref(false)
 
-const calculateFloatingPosition = async () => {
-  const { x, y } = await computePosition(refReference.value.refInputContainer, refFloating.value, {
-    placement: 'bottom-start',
-    middleware: [
-      offset(6),
-      {
-        name: 'sameWidth',
-        fn: ({ rects, x, y }) => {
-          // Set width of reference to floating
-          refFloating.value.style.width = `${rects.reference.width}px`
-
-          return { x, y }
-        },
-      },
-      flip(),
-      shift({ padding: 10 }),
-    ],
-  })
-
-  Object.assign(refFloating.value.style, {
-    left: `${x}px`,
-    top: `${y}px`,
-  })
-}
-
-let floatingUiCleanup: Function = () => { }
-onMounted(() => {
-  nextTick(() => {
-    floatingUiCleanup = autoUpdate(refReference.value.refInputContainer, refFloating.value, calculateFloatingPosition)
-  })
-})
-
-onBeforeUnmount(() => floatingUiCleanup())
-
 onClickOutside(
   refFloating,
   _event => {
@@ -104,27 +67,24 @@ const handleInputClick = () => {
 }
 
 // 👉 Options
-const optionClasses = 'a-select-option states before:transition-none cursor-pointer text-ellipsis overflow-hidden'
-const handleOptionClick = (option: SelectOption) => {
-  const value = isObjectOption(option) && !props.emitObject ? (option as ObjectOption).value : option
-
-  emit('change', value)
-  emit('input', value)
-  emit('update:modelValue', value)
+const handleOptionClick = (item: ListItemProps, value: any) => {
+  const valueToEmit = props.emitObject ? item : value
+  emit('change', valueToEmit)
+  emit('input', valueToEmit)
+  emit('update:modelValue', valueToEmit)
 }
 const closeOptions = (event: MouseEvent) => {
   if (event.target !== refFloating.value)
     isOptionsVisible.value = false
 }
 
-// 👉 Value
-const selectedValue = computed(() => {
-  const option = props.options.find(option => isObjectOption(option)
-    ? (option as ObjectOption).value === (!props.emitObject ? props.modelValue : (props.modelValue as ObjectOption).value)
-    : option === props.modelValue)
-
-  return option ? isObjectOption(option) ? (option as ObjectOption).label : option : (props.modelValue as ObjectOption | undefined)?.label || ''
-})
+// 👉 Middleware
+const middleware = () => [
+  offset(6),
+  sameWidthFloatingUIMiddleware(refFloating),
+  flip(),
+  shift({ padding: 10 }),
+]
 </script>
 
 <template>
@@ -155,33 +115,32 @@ const selectedValue = computed(() => {
         ref="selectRef"
         readonly
         class="a-select-input"
-        :value="selectedValue"
+        :value="isObject(modelValue) ? modelValue.text : modelValue"
       >
     </template>
   </ABaseInput>
 
   <!-- 👉 Select options -->
-  <Teleport
-    v-if="isMounted"
-    :to="teleportTarget"
+  <AFloating
+    :reference-el="refReference && refReference.refInputContainer"
+    :middleware="middleware"
+    class="a-select-floating"
   >
-    <ul
+    <ACard
       v-show="isOptionsVisible"
       ref="refFloating"
-      class="a-select-options-container absolute bg-[hsl(var(--a-surface-c))]"
+      :data-slots="Object.keys($slots)"
+      class="a-select-options-container bg-[hsl(var(--a-surface-c))]"
       :class="props.optionsWrapperClasses"
       @click="closeOptions"
     >
-      <slot :attrs="{ class: optionClasses }">
-        <li
-          v-for="(option, index) in props.options"
-          :key="index"
-          :class="optionClasses"
-          @click="handleOptionClick(option)"
-        >
-          {{ isObjectOption(option) ? (option as ObjectOption).label : option }}
-        </li>
-      </slot>
-    </ul>
-  </Teleport>
+      <AList
+        :items="options"
+        :value="props.modelValue"
+        class="a-select-options-list"
+        :class="props.listClasses"
+        @click:item="({ item, value }) => handleOptionClick(item, value)"
+      />
+    </ACard>
+  </AFloating>
 </template>
diff --git a/packages/preset-theme-default/src/scss/index.scss b/packages/preset-theme-default/src/scss/index.scss
index bd84ac6b..67574c11 100644
--- a/packages/preset-theme-default/src/scss/index.scss
+++ b/packages/preset-theme-default/src/scss/index.scss
@@ -12,6 +12,8 @@ $body-color: hsla(var(--a-base-c), 0.68);
     // #region all-css-vars
     --a-base-c: 0, 10%, 20%;
 
+    --a-spacing: 1;
+
     // ℹ️ Represents theme color. light => white, dark => black. ATM, it's used for backdrop & badge outline color.
     --a-theme-c: 0, 0%, 100%;
 
@@ -69,11 +71,11 @@ $body-color: hsla(var(--a-base-c), 0.68);
     // 👉 List
     // ℹ️ We might not need `--a-list-gap` this if we are using just `AList`, however when we use list inside another component this will be helpful
     // #region a-list-css-vars
-    --a-list-gap: 0rem;
-    --a-list-item-gap: 0.75rem;    
-    --a-list-item-padding: 0.5rem 1rem;
-    --a-list-item-margin: 0rem;
-    --a-list-item-min-height: 2.5rem;
+    --a-list-gap: calc(0em * var(--a-spacing));
+    --a-list-item-gap: calc(0.75em * var(--a-spacing));
+    --a-list-item-padding: calc(0.5em * var(--a-spacing)) calc(1em * var(--a-spacing));
+    --a-list-item-margin: calc(0em * var(--a-spacing));
+    --a-list-item-min-height: calc(2.5em * var(--a-spacing));
     // #endregion a-list-css-vars
 
     // #endregion all-css-vars
diff --git a/packages/preset-theme-default/src/shortcuts.ts b/packages/preset-theme-default/src/shortcuts.ts
index 017afb92..3be50dd5 100644
--- a/packages/preset-theme-default/src/shortcuts.ts
+++ b/packages/preset-theme-default/src/shortcuts.ts
@@ -130,8 +130,9 @@ const shortcuts: Exclude<Preset['shortcuts'], undefined | StaticShortcutMap> = [
     'a-radio-disabled': 'opacity-50',
 
     // 👉 Select
-    'a-select-options-container': 'z-10 border border-solid border-a-border em:spacing:rounded-lg em:spacing:py-3 shadow-lg',
-    'a-select-option': 'em:spacing:px-4 em:spacing:py-1',
+    'a-select-floating': '[--a-transition-slide-up-transform:6px]',
+    'a-select-options-container': '',
+    'a-select-options-list': '![--a-spacing:0.75]',
 
     // 👉 Switch
     'a-switch': 'select-none',