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

`backstack.removeAllStateChangeCompletionListeners()` removes the scope activation dispatch completion listener #221

Closed Zhuinden closed 4 years ago

Zhuinden commented 4 years ago

In 2.3.1, I fixed a bug by registering the managed state changer's second half as a completion listener.

However, invoking removeAllStateChangeCompletionListeners() will remove this listener as well.

I should prevent that from ever happening.

Zhuinden commented 4 years ago

Fixed in https://github.com/Zhuinden/simple-stack/commit/4bf2712d153b2f770e609aecce00df9dee940f4a