preactjs / preact / 5718

100%

master: 100%

push

travis-ci

Properly flush nested renderCallbacks (-3 B)

Previously, we would synchronously flush nested renderCallbacks. This behavior meant that nested setState calls would flush before there state was properly applied in diff, meaning DOM, props, etc. wouldn't have been updated for that setState call when the callback was run (though the state would be correct due to a shortcut we took).

This PR fixes this behavior by only invoking the renderCallbacks for the current setState call synchronously. Nested setState callbacks are queued and executed when those setState cals are applied.

822 of 845 branches covered (97.28%)

855 of 857 relevant lines covered (99.77%)

788.24 hits per line