Miguel Vargas

Good observation. I don't understand the use of FeatureScope at all since the components are created in the inject methods which are called from each Activity's onCreate, which means they...

Also add Build->Clean to the unconfigurable IDE commands. I noticed that Intellij with Android plugin does not have the Clean command, so it must be an Android Studio specific thing.

Any chance you could create a minimal project that demonstrates the bug?

Yes it's a leak. BottomNavigator will create new fragments and ignore the existing ones inside FragmentManager.

Considered several solutions and decided that in the interest of keeping BottomNavigator's API small we'll use androidx's Lifecycle Viewmodel Savedstate when it becomes available. Unfortunately that library is at version...

Moved the memory leak bug to Issue #4. This ticket is for the enhancement to preserve BottomNavigator state after process death.

I wonder if it would be possible to use the advanced Mockito APIs to intercept the call and add the extra matchers automatically. https://static.javadoc.io/org.mockito/mockito-core/2.23.4/org/mockito/Mockito.html#framework_integrations_api

Is there any justification for `Leave other operators before the wrap:`? I find it more readable to put boolean operators at the beginning of a long chain because it highlights...