zwave-js / zwave-js-ui / 22755423550

18%

push

github

feat: make the network graph truly dynamic (#4573)

This PR fixes issues with the network graph that have bugged me for a
looooong time. It removes the `live view` toggle and `reload graph`
button, because they are now no longer necessary.

Summary of changes:
- Node status (dead/alive) updates immediately, not just after a reload
- Removing a node closes the details view and removes it from the
network graph
- Adding a node adds it to the network graph, along with its primary
route
- Changing routes, adding and removing nodes no longer causes the entire
network map to re-layout. Physics is only enabled for a short time after
changes to incrementally update the layout.
- Updating statistics (e.g. RSSI, speed, ...) in the details view only
updates the edge color and annotation and no longer re-layouts it.

---------

Co-authored-by: Daniel Lando <daniel.sorridi@gmail.com>

334 of 444 branches covered (75.23%)

Branch coverage included in aggregate %.

0 of 5 new or added lines in 1 file covered. (0.0%)

4054 of 23524 relevant lines covered (17.23%)

1.07 hits per line