aragon / aragon-apps / 3996

97%

push

travis-ci-com

Voting: skip unnecessary renderings (#829)

Noticeable improvement: the side panels transition should be smoother.

Changes:

- Move panels in their own components, so that the rendering of
  SidePanel can be skipped entirely.

- Memoize the object returned by usePanel(), so that it can be compared
  using a strict equality.

- Move the voting panel actions in their own component fetching their
  data, so that the panel component doesn’t need to render that much
  anymore.

- Don’t recreate `votes` every second, instead only do it if any
  “opened” status has changed.

120 of 128 branches covered (93.75%)

Branch coverage included in aggregate %.

270 of 274 relevant lines covered (98.54%)

34.35 hits per line