fix(Select): fix render when using virtual scroll with custom label (#…

…4677) * fix(Select): fix label render when using virtual scroll with custom label * chore: fix before unmount * fix(select): fix filterable behavior when pressing enter * chore: comment
Tencent · Oct 26, 2024 · e933f42 · e933f42
1 parent d556760
commit e933f42
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/src/select-input/select-input.tsx b/src/select-input/select-input.tsx
@@ -86,7 +86,7 @@ export default defineComponent({
     });
 
     onBeforeUnmount(() => {
-      selectInputRef.value.removeEventListener('keydown', addKeyboardEventListener);
+      selectInputRef.value?.removeEventListener('keydown', addKeyboardEventListener);
     });
 
     const onOverlayClick: PopupProps['onOverlayClick'] = (ctx) => {

diff --git a/src/select/hooks/useKeyboardControl.ts b/src/select/hooks/useKeyboardControl.ts
@@ -136,7 +136,7 @@ export default function useKeyboardControl({
   watch(hoverIndex, (index) => {
     const optionHeight = selectPanelRef.value?.innerRef?.querySelector(
       `.${classPrefix.value}-select-option`,
-    ).clientHeight;
+    )?.clientHeight;
 
     const scrollHeight = optionHeight * index;
 

diff --git a/src/select/option.tsx b/src/select/option.tsx
@@ -124,9 +124,6 @@ export default defineComponent({
         trigger: val ? 'check' : 'uncheck',
         e: context.e,
       });
-      if (!selectProvider.value.reserveKeyword) {
-        selectProvider.value.handlerInputChange('');
-      }
     };
 
     const renderTitle = () => {

diff --git a/src/select/select-panel.tsx b/src/select/select-panel.tsx
@@ -81,7 +81,7 @@ export default defineComponent({
                       scrollType: props.scroll?.type,
                       isVirtual: isVirtual.value,
                       bufferSize: props.scroll?.bufferSize,
-                      key: `${item.$index || ''}_${index}`,
+                      key: `${item.$index || ''}_${index}_${item.value}`,
                     }
                   : {
                       key: `${index}_${item.value}`,

diff --git a/src/select/select.tsx b/src/select/select.tsx
@@ -87,6 +87,7 @@ export default defineComponent({
         newVal = props.multiple ? (newVal as SelectValue[]).map((val) => getOption(val)) : getOption(newVal);
       }
       if (newVal === orgValue.value) return;
+      if (props.multiple && !props.reserveKeyword) setInputValue('');
       setOrgValue(newVal, {
         selectedOptions: getSelectedOptions(newVal),
         ...context,
@@ -167,7 +168,8 @@ export default defineComponent({
     const handleCreate = () => {
       if (!innerInputValue.value) return;
       props.onCreate?.(innerInputValue.value);
-      setInputValue('');
+      // only clean input value when reopen popup
+      if (!innerPopupVisible.value) setInputValue('');
     };
 
     const popupContentRef = computed(() => selectInputRef.value?.popupRef.getOverlay() as HTMLElement);