Skip to content

Add: shouldCloseEditor prop#2722

Closed
blancham wants to merge 10 commits intoComcast:mainfrom
kraaft-co:main
Closed

Add: shouldCloseEditor prop#2722
blancham wants to merge 10 commits intoComcast:mainfrom
kraaft-co:main

Conversation

@blancham
Copy link
Copy Markdown

Context

React-data-grid works great in a local environment but when it comes to use it with a database, problems appear...

Imagine you have 2 tables on two different laptops. With the current implementation, if I'm editing a cell and that the other guy is modifying the grid data, my cell editor will be closed automatically independently of what row/ cell has been edited.

Solution

shouldCloseEditor is a function and allows you to specify the behaviour when you are editing a cell and that new data comes from the outside of the table.

In my project, I close the editor only when the value of the cell I'm currently editing has changed.

@amanmahajan7
Copy link
Copy Markdown
Collaborator

Thank you for the PR. We have added a new prop to handle this scenario column.editorOptions.closeOnExternalRowChange
#3759

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants