Re-render on steam/subscription handlers (just like EventHandlers).

[rebo]

EventHandlers currently re-render if a message is not returned by the event-handler.

However this is not currently mirrored by Stream and Subscription handlers. I.e. if a handler does not return a message seed currently does not re-render.

I feel this should be the same as EventHandler. Unless there are any objections I can submit a PR to this effect.

For instance the impact on this is for Stream/Subscription handlers I am having to use a Msg:NoOp.

No biggie but would be good to be consistent.

[MartinKavik]

I think that in

seed/src/app.rs

Lines 559 to 567 in 3921466

	pub fn mailbox(&self) -> Mailbox<Ms> {
	Mailbox::new(enclose!((self => s) move |option_message| {
	if let Some(message) = option_message {
	s.update(message);
	} else {
	s.rerender_vdom();
	}
	}))
	}

s.rerender_vdom() should be replaced with self.schedule_render() to sync with animation frame loop - it's best practice and maybe it helps with your problems. Or do we have a reason why we want to force rerender?

[MartinKavik]

I did some changes in #460. mailbox from the previous comment has been also changed.

All handlers should schedule rerender and they should be invoked in a predictable order now.
@rebo Could you test it with your code and provide more info about problems or close the issue, please?

[MartinKavik]

No activity for months and it should be resolved, closing.

@rebo Please reopen when you find any related problems.