-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Improve dropdown draggable list #1738
chore: Improve dropdown draggable list #1738
Conversation
front/src/modules/ui/menu-item/components/MenuItemDraggable.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CSS change on dragging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, this will greatly improve the API of twenty by providing draggable lists
front/src/modules/ui/draggable-list/components/DraggableItem.tsx
Outdated
Show resolved
Hide resolved
front/src/modules/ui/draggable-list/components/DraggableItem.tsx
Outdated
Show resolved
Hide resolved
front/src/modules/ui/menu-item/components/MenuItemDraggable.tsx
Outdated
Show resolved
Hide resolved
front/src/modules/ui/menu-item/components/MenuItemDraggable.tsx
Outdated
Show resolved
Hide resolved
front/src/modules/ui/view-bar/components/ViewFieldsVisibilityDropdownSection.tsx
Show resolved
Hide resolved
background: isDragged | ||
? theme.background.transparent.light | ||
: 'none', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found the css fix on DraggableItem
@AdityaPimpalkar Thanks a lot, I have made a few small changes after discussing with the team here: #1874 |
closes: #1668
Feedback is appreciated! :)