Combine multiple state updates into a single operation to reduce renders about inquirer.js HOT 2 CLOSED

Hi! Thanks for bringing that feedback

I want to batch renders to handles cases like what you outlined. I didn't have time yet to implement, but it's definitively an update the would make sense.

The code is over here if you want to give it a try. The tests should catch any regression you'd introduce (I think they're end-to-end enough to not bulk on delayed re-renders; but might need to review that too.)

from inquirer.js.

Implemented a naïve version of this.

So on any sync functions, the state updates will be merged (See code at 165d6d8)

But it won't work if calling multiple times setState from within an async callback (like setTimeout.) Handling those would require to have asynchronous rendering loops, and that makes everything more complicated (testing, but also just things like making sure there's no delays when typing.) Since updating multiple states from async functions are not a pattern I've encountered often creating prompts, I'm leaning on the side of simplicity for now.

(And the code demo using an object state is a good work around for custom prompt requiring multi state updates from an async callback.)

from inquirer.js.