search

Zhuinden / simple-stack

[ACTIVE] Simple Stack, a backstack library / navigation framework for simpler navigation and state management (for fragments, views, or whatevers).
Apache License 2.0
1.36k stars 76 forks source link

Consider killing BackstackDelegate entirely #192

Closed Zhuinden closed 4 years ago

Zhuinden commented 5 years ago

BackstackDelegate was created to wrap the original Backstack class that is now the NavigationCore and internal.

The new Backstack class (extracted from BackstackDelegate's internals and eventually called BackstackManager) is actually easier to work with directly than through the BackstackDelegate now.

So it might make sense to just remove the BackstackDelegate from the library, and show a migration guide for how you need to use a Backstack in its place. This also solves the "onRetainCustomNonConfigurationInstance is deprecated" problem by handing over the responsibility of preserving across config changes to the library users 😉

Zhuinden commented 4 years ago

No need to track this issue, depreciation tracks it already. I won't forget.