preactjs / preact / 5815

100%

push

travis-ci

Rewrite useImperativeHandle to use useLayoutEffect (-35 B) (#2003)

* Rewrite useImperativeHandle to use useLayoutEffect (-41 B)

* Add new useContext test

* Rerun imperative handler creator if ref changes (+3 B)

* Improve hooks JSDoc

* Assign the return value of push to _afterPaintQueued (-2 B)

* Consistenly assign a number to _afterPaintQueued (-1 B)

* Support no deps array in useImperativeHandle (+12 B)

* Rename `safeRaf` to `afterNextFrame`

...since its implementation does more than a real `safeRaf` implementation. In other words, `safeRaf` invokes the callback after a rAF (or timeout) and another timeout (line 241) so its not a pure `safeRaf`. So to avoid confustion, I renamed the function to be clearer and added a comment describing its purpose

* Reuse `prevRaf` variable (-4 B)

822 of 847 branches covered (97.05%)

857 of 860 relevant lines covered (99.65%)

771.92 hits per line