search

staltz / rxmarbles

Interactive diagrams of Rx Observables
http://www.rxmarbles.com
BSD 3-Clause "New" or "Revised" License
4.21k stars 536 forks source link

skip/skipLatest are messed up #6

Closed machty closed 9 years ago

machty commented 9 years ago

Wish I could provide a better description of the error but here's a visual:

skip/skipLatest

It seems fine when refreshed, but if you navigate elsewhere and come back, it gets screwed up and displays some other result (not sure what).

kolree commented 9 years ago

I am seeing this same issue - related to the problem with the diagram for "First" I tweeted you about. This seems to be a more general issue with the refresh somehow. To reproduce:

  1. view Pausable Buffered
  2. now try and view other diagrams - the results row seems to be stuck on the results from Pausable Buffered I am using Safari 8.0.2 on OS X 10.10.1. I also see the same behavior on IE 11 on my Windows 7 VM.
staltz commented 9 years ago

Thanks guys, this is a known issues. I am in the process of migrating RxMarbles to Cycle.js, and that will fix a number of issues. This issue here will be fixed by using the new withLatestFrom in these lines: https://github.com/staltz/rxmarbles/blob/master/src/models/sandbox-output.js#L83

staltz commented 9 years ago

Fixed in v1.2.1