From c99a130fdce90fccecc80071409082d01e414d3a Mon Sep 17 00:00:00 2001 From: David Matejka Date: Fri, 24 Nov 2023 13:50:35 +0100 Subject: [PATCH 1/3] fix(react-choice-field): do not send onMove when sortableBy is empty --- .../hooks/useDynamicMultipleChoiceWithConnectingEntityField.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/react-choice-field/src/hooks/useDynamicMultipleChoiceWithConnectingEntityField.ts b/packages/react-choice-field/src/hooks/useDynamicMultipleChoiceWithConnectingEntityField.ts index 472bd0905e..5dedbde6be 100644 --- a/packages/react-choice-field/src/hooks/useDynamicMultipleChoiceWithConnectingEntityField.ts +++ b/packages/react-choice-field/src/hooks/useDynamicMultipleChoiceWithConnectingEntityField.ts @@ -69,7 +69,7 @@ export const useDynamicMultipleChoiceWithConnectingEntityField = ( onRemove: useCallback(value => { optionIdToConnectingEntityMap.get(value.id)?.deleteEntity() }, [optionIdToConnectingEntityMap]), - onMove: sortedConnectingEntities.moveEntity, + onMove: props.sortableBy ? sortedConnectingEntities.moveEntity : undefined, onSearch, isLoading, } From 4b4759c97d72b1ea872045935fe5ddd04cedde19 Mon Sep 17 00:00:00 2001 From: David Matejka Date: Fri, 24 Nov 2023 13:51:17 +0100 Subject: [PATCH 2/3] fix(react-choice-field-ui): remove portal provider and fixed position, the scroll was broken --- .../src/components/rendering/MultiSelectFieldRenderer.tsx | 2 +- .../src/components/rendering/SelectFieldRenderer.tsx | 1 - .../src/hooks/useCommonReactSelectProps.ts | 1 - 3 files changed, 1 insertion(+), 3 deletions(-) diff --git a/packages/react-choice-field-ui/src/components/rendering/MultiSelectFieldRenderer.tsx b/packages/react-choice-field-ui/src/components/rendering/MultiSelectFieldRenderer.tsx index 45b6d6dc64..d32a8f8f43 100644 --- a/packages/react-choice-field-ui/src/components/rendering/MultiSelectFieldRenderer.tsx +++ b/packages/react-choice-field-ui/src/components/rendering/MultiSelectFieldRenderer.tsx @@ -140,7 +140,7 @@ export const MultiSelectFieldInner = memo(({ MultiValueLabel: SortableMultiValueLabel, } as any} /> - : } diff --git a/packages/react-choice-field-ui/src/components/rendering/SelectFieldRenderer.tsx b/packages/react-choice-field-ui/src/components/rendering/SelectFieldRenderer.tsx index 13b94b19b1..bdf1ecb6f4 100644 --- a/packages/react-choice-field-ui/src/components/rendering/SelectFieldRenderer.tsx +++ b/packages/react-choice-field-ui/src/components/rendering/SelectFieldRenderer.tsx @@ -98,7 +98,6 @@ export const SelectFieldInner = typedMemo(({