search

glimmerjs / glimmer-blueprint

[MOVED] This package is now part of the Glimmer.js monorepo
https://github.com/glimmerjs/glimmer.js
25 stars 13 forks source link

Prevent initial render + double rerender #20

Closed pittst3r closed 7 years ago

pittst3r commented 7 years ago

Before:

screen shot 2017-03-31 at 11 20 56 pm

After:

screen shot 2017-03-31 at 11 19 35 pm

cc @GavinJoyce @chadhietala re: https://github.com/glimmerjs/glimmer-application/issues/43#issuecomment-290741098

GavinJoyce commented 7 years ago

Nice one, I've confirmed that this fixes the double rerender on boot

dgeb commented 7 years ago

Thanks @robbiepitts!

This fix feels brittle, but is clearly better than the status quo. I've just published this in v0.1.13.

pittst3r commented 7 years ago

@dgeb For sure, just wanted to take care of this low-hanging fruit. 😃