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.

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

[data grid] Implement Row reorder #206

Closed
oliviertassinari opened this issue Aug 22, 2020 · 24 comments · Fixed by #4034
Closed

[data grid] Implement Row reorder #206

oliviertassinari opened this issue Aug 22, 2020 · 24 comments · Fixed by #4034
Assignees
Labels
component: data grid This is the name of the generic UI component, not the React module! feature: Reordering Related to the data grid Reordering feature linked in docs The issue is linked in the docs, so completely skew the upvotes new feature New feature or request plan: Pro Impact at least one Pro user

Comments

@oliviertassinari
Copy link
Member

oliviertassinari commented Aug 22, 2020

Benchmark

@oliviertassinari oliviertassinari added component: data grid This is the name of the generic UI component, not the React module! new feature New feature or request labels Aug 22, 2020
@oliviertassinari oliviertassinari added the linked in docs The issue is linked in the docs, so completely skew the upvotes label Nov 30, 2020
@LincMitch

This comment was marked as off-topic.

@oliviertassinari oliviertassinari changed the title [DataGrid] Implement Row reorder [XGrid] Implement Row reorder Jul 7, 2021
@oliviertassinari oliviertassinari added components: XGrid and removed component: data grid This is the name of the generic UI component, not the React module! labels Jul 7, 2021
@Fasani
Copy link

Fasani commented Jul 18, 2021

The company I work for use Material UI, and we need this feature.

I have two options. I guess I can have one team implement https://github.com/Autodesk/react-base-table and style it or have them complete this task for the MUI library.

If we were to add this feature, what would the process be, is there contribution forms that need to be filled in and signed, or do we just fork and open a PR.

This page is a bit bare in terms of requirements/spec, is there more info available, or can we just implement it how we see fit for our needs.

Sorry if these topics are covered somewhere. Please point me to a link if so. Thank you.

@oliviertassinari
Copy link
Member Author

oliviertassinari commented Jul 18, 2021

@Fasani Hey, thanks for the interest in contributing features into the library. This case has come up a couple of times in the past. For Pro feature, what we generally propose is as follow:

  • if the changes gets in, we provide licenses free of charges.
  • in order to accept the changes, we need to sign a copyright assignment agreement.

You can sent an email at x@mui.com if you want to continue the conversation.

@Fasani
Copy link

Fasani commented Jul 18, 2021

Thanks for getting back to me.

I noticed this is on DataGrid and perhaps I would want it on Table. Does that make a difference?

Likely I will check out the code and start considering the change. If it makes sense I can send it back to you.

@flaviendelangle
Copy link
Member

flaviendelangle commented Jul 21, 2021

I doubt it will ever be implemented in the Table component.
What would stop you to migrate from Table to DataGrid?

@Fasani
Copy link

Fasani commented Jul 22, 2021

I ended up going with option A, thanks for the replies.

@gyudisanyi

This comment has been minimized.

@oliviertassinari oliviertassinari changed the title [XGrid] Implement Row reorder [DataGridPro] Implement Row reorder Aug 30, 2021
@camnealie
Copy link

I note there's a feature request (mentioned in OP) for draggable rows to be implemented in table, has anyone made any progress for this in datagrid / datagrid pro? Would be nice!

@flaviendelangle
Copy link
Member

Not yet no
We are focusing on other features that seems to be wanted by more people.
But I think the row reorder should not be a major pain to implement when we will have the time for it.

@oliviertassinari oliviertassinari added the plan: Pro Impact at least one Pro user label Dec 5, 2021
@trentschnee

This comment has been minimized.

@paguerrie
Copy link

This feature would convince me to upgrade to the pro plan.

@marc-polizzi

This comment has been minimized.

@david-ic3

This comment has been minimized.

@DanailH DanailH added component: data grid This is the name of the generic UI component, not the React module! and removed component: DataGridPro labels Jan 25, 2022
@Snosii
Copy link

Snosii commented Feb 2, 2022

any idea, whether this feature would be available in Q1 this year?

@adasko86
Copy link

adasko86 commented Feb 3, 2022

Our company is also waiting for this feature. Do you have any plans for that?

@DanailH DanailH self-assigned this Feb 7, 2022
@innocent-maina
Copy link

This would work really well for my project, I have so many rows on particular tables, re-ordering them would make analysis much easier

@DanailH DanailH mentioned this issue Feb 25, 2022
6 tasks
@oliviertassinari oliviertassinari changed the title [DataGridPro] Implement Row reorder [data grid] Implement Row reorder Mar 16, 2022
@oriolcx
Copy link

oriolcx commented Apr 11, 2022

We have a couple of licenses and we would also be very interested in this feature.

@Mistes974
Copy link

Hi ! Why is it taking so long to merge ? We are looking forward to this feature.

@DanailH
Copy link
Member

DanailH commented Apr 20, 2022

I apologize for the delay. I will resume the work on it today and aim to have it in this week's release.

@mohamedsendii
Copy link

Hello, Does row reorder work with DataGrid tree data?

@flaviendelangle
Copy link
Member

For now it does not
If there is enough traction, we might implement it later with callbacks to update your data accordingly.

@DanailH maybe we could create a new issue "Row reorder for Row Grouping and Tree Data", put it "waiting for upvote" and link it in the doc.

@mohamedsendii
Copy link

yes for example to implement a hierarchy of an organization in a Datagrid tree data it will be practical to do the drag and drop of rows in order to modify the position of the nodes in this organization without going through a form, I hope that you create a new issue "Row reorder for Row Grouping and Tree Data"

@gustavozirbes
Copy link

For now it does not If there is enough traction, we might implement it later with callbacks to update your data accordingly.

@DanailH maybe we could create a new issue "Row reorder for Row Grouping and Tree Data", put it "waiting for upvote" and link it in the doc.

Hi @flaviendelangle. Is there a clear path of what are the priorities for the data grid? I feel that things are being added without much planning in terms of closing the gaps.

For instance: the tree functionality was added but without support for lazy load, which makes it pretty hard to be used in enterprise applications. Now we have the row reorder, but it does not work with tree data.

Please don't take me wrong, I really like working with MUI components, but it is been hard to use these new features where one thing does not (fully) work with another. When I look at the roadmap, I can't see these gaps getting closed.

@flaviendelangle
Copy link
Member

For the row reorder, I think the usage with a tree structure is far from being the majority of the use cases.
So we decided that it was better to launch the feature for all the people not using a tree structure rather than delaying.
With that being said, I agree that it is easy to forget to "close the gaps" as you say.

For the lazy loading of the tree. Is the custom example enough for you ?
And if not, what is missing ?
We will have to implement it correctly on our side at some point. But it requires to rework our entire internal data model and we decided to go for others features before that felt easier to do relative to the community traction.

@oliviertassinari oliviertassinari added the feature: Reordering Related to the data grid Reordering feature label Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! feature: Reordering Related to the data grid Reordering feature linked in docs The issue is linked in the docs, so completely skew the upvotes new feature New feature or request plan: Pro Impact at least one Pro user
Projects
None yet
Development

Successfully merging a pull request may close this issue.