-
Notifications
You must be signed in to change notification settings - Fork 630
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
Incorrect height of scrolling element after reinitialization. #351
Comments
Please make more informative why I should do that or PR with more description |
Hello. jScrollPane/script/jquery.jscrollpane.js Line 184 in ae5dc1e
In this case, the element will again have the same dimensions as before the initialization. |
Is there possibility to make PR so I can see all needed actions and some test case? |
Actually it's all ready done in line 156 and line 184 should be removed altogether. |
Hello, sorry, I did not answer for a long time. TEST CASE (tested on google chrome): 2. delete one or more columns by clicking on the "Remove a column" 3. click on "Update scroller" The width of the .jspPane must match the width of the table. |
Thank you very much! excellent example. I take a look at this. |
You are welcome. |
Sorry it's PR is Pull Request. |
Thanks |
I have a simular issue with de height of the container when implementing live search when rows disappear! Any ideas on how to fix this? |
As my time is limited you can audit that test case if it's the same as yours. If it's not you can submit small example how you example is different |
Have new version made any progress on this? |
Hello @vitch , |
@RajuTicvic does commented fix your situation? |
Which comments |
This first one which @alexeylepesin tells you to change place in code or is your problem a new Issue? |
No |
@RajuTicvic Actually your request doesn't have anything to do with jScrollPane as it's purpose is only to 'skin' your scrollbars not alter how HTML tables work. |
jScrollPane/script/jquery.jscrollpane.js
Line 185 in ae5dc1e
Please move line 185 to 162
The text was updated successfully, but these errors were encountered: