Skip to content

Commit 284e757

Browse files
Fixes Strange behaviour when reordering options of select field (#6103) (#6339)
fixes #6103 --------- Co-authored-by: Marie Stoppa <[email protected]>
1 parent f24536c commit 284e757

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

packages/twenty-front/src/modules/settings/data-model/fields/forms/select/components/SettingsDataModelFieldSelectForm.tsx

+1
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,7 @@ export const SettingsDataModelFieldSelectForm = ({
218218
<>
219219
{options.map((option, index) => (
220220
<DraggableItem
221+
isInsideScrollableContainer
221222
key={option.id}
222223
draggableId={option.id}
223224
index={index}

packages/twenty-front/src/modules/ui/layout/draggable-list/components/DraggableItem.tsx

+3-1
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,15 @@ type DraggableItemProps = {
77
isDragDisabled?: boolean;
88
index: number;
99
itemComponent: JSX.Element;
10+
isInsideScrollableContainer?: boolean;
1011
};
1112

1213
export const DraggableItem = ({
1314
draggableId,
1415
isDragDisabled = false,
1516
index,
1617
itemComponent,
18+
isInsideScrollableContainer,
1719
}: DraggableItemProps) => {
1820
const theme = useTheme();
1921
return (
@@ -36,7 +38,7 @@ export const DraggableItem = ({
3638
style={{
3739
...draggableStyle,
3840
left: 'auto',
39-
top: 'auto',
41+
...(isInsideScrollableContainer ? {} : { top: 'auto' }),
4042
transform: draggableStyle?.transform?.replace(
4143
/\(-?\d+px,/,
4244
'(0,',

0 commit comments

Comments
 (0)