-
Notifications
You must be signed in to change notification settings - Fork 347
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
autocommit changes when begin editing another row from an active edit… #1015
base: master
Are you sure you want to change the base?
Conversation
… event occur such a document location change ( update can be invoke through jg.jsGrid("updateItem") )
… changes using jsGrid("editPending") and confirm programmatically by jsGrid("confirmPending"). This function returns false if some validation failed to allow you preventDefault() of your save button
…ow is toggled to avoid validation (if there are some default templated value in control that may cause validation fail even insert row off)
…nder cells, useful for total row display
src/jsgrid.core.js
Outdated
@@ -1390,7 +1390,7 @@ | |||
return; | |||
|
|||
if(this._editingRow) { | |||
this.cancelEdit(); | |||
this._finishUpdate(this._editingRow, this._getEditedItem()); |
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.
This is what I have been looking for. Thank you
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.
do I need to edit the distro or can I override this method?
devel0 I would like to make a pull request from your rep for these changes. But your repo is no longer public. would you please share with me? |
Hi Barry, may repo changed into follow name |
autocommit changes when begin editing another row from an active editing row ( would fix #1011 )