MithrilJS / mithril.js

A JavaScript Framework for Building Brilliant Applications
https://mithril.js.org
MIT License
13.98k stars 925 forks source link

mithril stream .map does not trigger next stream value #2643

Open foxdonut opened 3 years ago

foxdonut commented 3 years ago

Mithril version:

2.0.4

Browser and OS:

Chrome on Mac OSX

Project:

Meiosis

Steps to Reproduce

Expected Behavior

Current Behavior

Context

This is to set up the Meiosis pattern with Effects.

Note: This commit fixes the same issue in Meiosis-Setup's simple-stream.

pygy commented 3 years ago

@foxdonut have you investigated how to fix this?

foxdonut commented 3 years ago

@pygy yes, see the last line of my description above, there is a link to a commit showing how I fixed it in simple-stream. I figure the fix should be similar in mithril stream.

pygy commented 3 years ago

Thanks, I'll have to dig into that then :-) (unless you beat me to a PR)

ericop commented 3 years ago

@pygy here is a working flems based on @foxdonut example. I tried making a PR, but with other changes in the next branch were beyond my wizardry to make play nice together.