CoordinatorBehaviorExample icon indicating copy to clipboard operation
CoordinatorBehaviorExample copied to clipboard

What's the value of maybeInitProperties()?

Open Jrodseth opened this issue 7 years ago • 1 comments

Why check if any of the private fields need to be initialized every time onDependentViewChanged is called? Do you expect the values to be reset to 0 by something?

@Override public boolean onDependentViewChanged(CoordinatorLayout parent, CircleImageView child, View dependency) { maybeInitProperties(child, dependency);

I would think initializing them once would be enough so I'm wondering if I'm missing something that requires this approach?

Jrodseth avatar May 11 '17 01:05 Jrodseth

Also, the mCustom... attributes are set from XML in the constructor, but never read?

drinfernoo avatar Mar 09 '18 05:03 drinfernoo