ReactiveX / rxjs

A reactive programming library for JavaScript
https://rxjs.dev
Apache License 2.0
30.65k stars 3k forks source link

combineLatest Marble Diagram not correct #6841

Open BingeCode opened 2 years ago

BingeCode commented 2 years ago

Describe the bug

The marble diagram in the combineLatest documentation is not correct. It does not seem to follow a specific pattern for how the output values are coloured and is confusing.

image

Expected behavior

The output values should always have the colour of the most recent step as in: a1: red b1: yellow b2: yellow b3: blue, ...

Or is there another pattern that I'm not seeing?

Reproduction code

No response

Reproduction URL

No response

Version

7.1.0

Environment

No response

Additional context

No response

Balastrong commented 2 years ago

I also find the colors confusing in the current diagram. Taking the color of the last emitted value makes much more sense.