jledentu / vue-finder

📁 A Vue.js component to display hierarchical data (like the MacOS X finder)
https://vue-finder.netlify.app
MIT License
160 stars 16 forks source link

perf: avoid update when expanding same node #207

Closed jledentu closed 1 year ago

jledentu commented 1 year ago

This PR early returns when update tries to expand the already expanded node

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (9455989) 99.42% compared to head (3b6d894) 99.43%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## 2.x #207 +/- ## ======================================= Coverage 99.42% 99.43% ======================================= Files 4 4 Lines 175 177 +2 Branches 44 45 +1 ======================================= + Hits 174 176 +2 Misses 1 1 ``` | [Impacted Files](https://app.codecov.io/gh/jledentu/vue-finder/pull/207?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=J%C3%A9r%C3%A9mie+Ledentu) | Coverage Δ | | |---|---|---| | [src/utils/tree-model.js](https://app.codecov.io/gh/jledentu/vue-finder/pull/207?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=J%C3%A9r%C3%A9mie+Ledentu#diff-c3JjL3V0aWxzL3RyZWUtbW9kZWwuanM=) | `99.20% <100.00%> (+0.01%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

github-actions[bot] commented 1 year ago

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

The release is available on:

Your semantic-release bot :package::rocket: