Expose InteractionState in Material components [168025711]

Fixed

Bug

Status Update

No update yet.

Description

lp...@google.com

created issue #1

Sep 8, 2020 04:20PM

Adding the ability to hoist InteractionState in all our components enables a few powerful usecases, including:

Customizing parameters in a particular visual state, such as when a component is Pressed
Allowing testing the behavior / appearance of a component in a particular state, such as screenshot testing how a component looks like when Pressed
Allowing a component's visual state to be controlled by another component, for example a list item that when pressed makes a RadioButton looked pressed as well

Comments

ap...@google.com <ap...@google.com> #2Sep 9, 2020 01:14PM

Also see issue in https://github.com/joreilly/PeopleInSpace (after similar removal of call to delay() in refresh())

lp...@google.com <lp...@google.com> Sep 17, 2020 02:33PM

Marked as fixed.

ap...@google.com <ap...@google.com> #3Sep 17, 2020 02:33PM

I suspect issue is somehow related to fact that without that delay() we're not recomposing with refreshing set to true....and end up showing CircularArrowIndicator in PullRefreshIndicator when we recompose with it set to false

Issue 168025711

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Sep 9, 2020 01:14PM

lp...@google.com <lp...@google.com> Sep 17, 2020 02:33PM

ap...@google.com <ap...@google.com> #3Sep 17, 2020 02:33PM

Add comment

Issue metadata