perf: ⚡️ Expand All and Collapse All render only once

frappe / datatable

The Missing Javascript Datatable for the Web

https://frappe.io/datatable

MIT License

1.02k stars 164 forks source link

Closed SaiFi0102 closed 5 years ago

SaiFi0102 commented 5 years ago

Currently ExpandAllNodes and CollapseAllNodes render for each and every parent node and can cause the browser to hang for large reports.

This PR improves the performance by first processing the metadata of the report and rendering only once at the end.

Also includes setTreeDepth() API function (I'll open a PR in frappe to handle initial_depth)

netchampfaris commented 5 years ago

:tada: This PR is included in version 1.12.3 :tada:

The release is available on:

Your semantic-release bot :package::rocket: