Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: update sort property #265

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: update sort property #265

wants to merge 1 commit into from

Conversation

Veragin
Copy link

@Veragin Veragin commented Dec 15, 2022

No description provided.

@bnabet
Copy link

bnabet commented Mar 25, 2023

Hello, is it a fix for sort option which doesn't work with ReactSortable ?

@Veragin
Copy link
Author

Veragin commented Apr 4, 2023 •
edited
Loading

Hi yes, but it should be probably done for all react properties ...
sort option was working, but it is set onMount and than it couldn't be changed (unless you do remount)

@Hemanth-Tinku
Copy link

Other solution : You can use 'disabled ',instead of 'sort' ,which almost works the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andresin87 andresin87 Awaiting requested review from andresin87 andresin87 is a code owner

@owen-m1 owen-m1 Awaiting requested review from owen-m1 owen-m1 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Veragin @bnabet @Hemanth-Tinku