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

inlineEditing Add row with grouping: true #319

Closed
aberel opened this issue Apr 15, 2017 · 4 comments
Closed

inlineEditing Add row with grouping: true #319

aberel opened this issue Apr 15, 2017 · 4 comments
Labels

Comments

@aberel
Copy link

aberel commented Apr 15, 2017

(version jqgrid-4.13.6)
Using option "grouping: true" in conjunction with inlineEditing, when you click the "ADD" button, the new row appears bellow the first group title, and all the grouped rows are messed up.
(images attached)

grid add 1
grid add 2

@OlegKi OlegKi added the bug label Apr 15, 2017
OlegKi added a commit that referenced this issue Apr 17, 2017
The code of `$.jgrid.parseDataToHtml` used in `addRowData` is oriented on processing of data inside of `readInput` only. The changes are just a workaround before more deap (and more nice) changes will be implemented. Thanks @aberel (https:/aberel) for reporting the problem. See [the issue](#319) for more details.
@OlegKi
Copy link
Member

OlegKi commented Apr 17, 2017

Thank you for reporting of the bug! I committed the fix just now. Please confirm that the new code fix the problem and close the issue finally.

I recommend you to reload the grid after adding the row to place the new row in the correct group. One should place .trigger("reloadGrid") inside of setTimeout(function () {/*here*/}, 0); to allow the adding of data will be finished before the reload will be started.

@aberel
Copy link
Author

aberel commented Apr 17, 2017

Oleg, your welcome! I will try the fix and tell if it works.
I always use to realod the grid after add or edit any row.
Regards.

@aberel
Copy link
Author

aberel commented Apr 17, 2017

Oleg, all the fixes you apply are on the last version of jqrid?
My problem is I'm in version 4.13.6 and don't want to upgrade version by now (I plan to upgrade in the future, but you know I have to test all my wrapper every time I upgrade version)
So, my only solution for these fixes is to copy the piece of code and edit my local jqgrid.js file, which is a bad idea, I know that :-(

@aberel
Copy link
Author

aberel commented Apr 17, 2017

I tried the patch and works ok!!

@aberel aberel closed this as completed Apr 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants