-
Notifications
You must be signed in to change notification settings - Fork 423
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
the new onAutosizeColumn throw an "e" not defined error when calling autosizeColumns() #264
Comments
ghiscoding
changed the title
the new onAutosizeColumn throw an "e" not defined error
the new onAutosizeColumn throw an "e" not defined error when calling autosizeColumns()
Jul 25, 2018
Same error here |
My bad. Cut and paste error. Fixed and re-released. Also, there was an autosize plugin #198, I assume that was a completely separate thing, as it seems to use fit-to-text-size. |
ghiscoding
added a commit
that referenced
this issue
Jul 26, 2018
added a demo to test it, manually that is, see the related PR #265, thanks for fixing the bug so quickly |
6pac
added a commit
that referenced
this issue
Jul 26, 2018
feat(demo): add demo for testing grid.autosizeColumns fn, ref #264
ghiscoding
added a commit
that referenced
this issue
Dec 20, 2018
alhoo
pushed a commit
to alhoo/SlickGrid
that referenced
this issue
Dec 22, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@6pac
it looks like the last version with the new
onAutosizeColumns
throw an errordefined on this line
I think
e
should be used at all in this trigger as it doesn't seem to exist anywhere inautosizeColumns()
function. I callautosize()
with all my grids and this seems to break the grid, I'll have to go back to previous version since my grid doesn't get displayed at all with this error thrownThe text was updated successfully, but these errors were encountered: