Conversation
* Modification for deploying on k8s * Small fix on dockerfile * Added CI workflow file * Ignore pip pinning in CI
add deferRender = true to improve the performance in older browsers by only creating the DOM nodes for the items that are currently being displayed on page 1. See https://datatables.net/manual/ajax
timarney
left a comment
There was a problem hiding this comment.
Works great. For this PR thinking we should only include the patch for the single file change in tables.js
obrien-j
left a comment
There was a problem hiding this comment.
rebase to get rid of already merged commit plz
Can you also profile this on a datastore of say 2000 records, to get an idea of improvement in load time plz.
|
Tested with a sandbox database 3314 records. Using BrowserStack + Sandbox DB to localhost so not super accurate but seems like for IE 11 this speeds things up by guessing around 8secs. The big thing in this case is it's not creating a bunch of DOM nodes that it doesn't need off the top. |
|
The K8s branch was merged #127 so you can rebase to master then make you one line update. |
|
Closing in favour of #129 -> same fix |
Add deferRender = true to improve the performance in older browsers by only creating the DOM nodes for the items that are currently being displayed on page 1.
See https://datatables.net/reference/option/deferRender