Maxime Jacquemin authored 3 years ago and Maxime Jacquemin committed 2 years ago

The code is clean, performing computation through promises without
trying to hide them in an ugly way. No more blinking, it is awesome. Now
we can actually work on improvements.

Maxime Jacquemin authored 3 years ago and Maxime Jacquemin committed 2 years ago

- The `useSelection` hooks did not memoized the returned `setSelection`
  function, leading to performance issues if a hook depended on it.

- The standard behavior for `usePromise` is to take a building function
  and dependencies, automatically memoized the constructed promise, and
  then doing as before. However, we keep a `usePromiseNoMemo` with the
  previous behavior. It seems needed for the `theme` handling.

- `useCache` new hook allows to add a cache to a given function. It also
  takes an optional serialization function for complex types for which
  standard equality does not provide the expected behavior.

Maxime Jacquemin authored 3 years ago and Maxime Jacquemin committed 2 years ago

We gotta handle the promises correctly.
There is some features to add.