-
Notifications
You must be signed in to change notification settings - Fork 23
Open
Description
We changed from atRest$ to state$ because some members of the core team had strong feelings about it being enumerated (I think they found atRest:true to be strange to reason about). We should revisit this decision:
- We have aggregators now that work on
true/false. Ifstate$is emitting enums, we have to manually mapstate$totrue/falseor can't use the aggregators. For instance, see the complexity in aggregatingtossable.state$ - Interactions often interact with state machines: for instance, whether a bottom sheet is open or closed.
state$makes more sense as a name for interaction-specific state than for activity.
Metadata
Metadata
Assignees
Labels
No labels