search

andrewdavey / immutable-devtools

Chrome Dev Tools custom formatter for Immutable-js values
BSD 3-Clause "New" or "Revised" License
661 stars 29 forks source link

Implement support for Seq #23

Open langpavel opened 8 years ago

langpavel commented 8 years ago

How Seq should be displayed?

Diggsey commented 8 years ago

It should display up to the first N values, followed by an ellipsis if the sequence continues.

langpavel commented 8 years ago

Ok, does anybody need this? :-) I'm also afraid of side effects. Can iterating of Seq have udesired side effects?

Diggsey commented 8 years ago

Iterating any of the containers could potentially have side-effects given the ability to define properties in JS. I think in this case it's unlikely given the nature of code using ImmutableJS (namely, very pure, functional code)

langpavel commented 8 years ago

I know, but Seq is generated by function unlike other structures backed by data..