-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
All screens should be modal except the main tab screens #507
Comments
Thanks for raising this point @vujevits I had to make some research and reconsider things I took as granted or didn't think through enough. I shared some examples in our Slack channel, but here are my main takeaways:
|
At the moment I find our navigation a bit complex, and I would enjoy using a simpler app. Let's just keep this aspect in mind, and continue the discussion if there are new ideas we could implement. |
oh yeah on that point I fully agree |
Right now it's easy to end up in the app in not thought through states of the navigation, the different tabs can be deep in a navigation stack, and going back to another tab leads you not to the tab's main screen, but to a place where something was opened previously (in arbitrary depth in a stack).
Also with every new feature where navigation happens as well (or adding a button for a screen to reach it from a new place), we have to consider if it changes the tab - this should not be an issue in this model.
The text was updated successfully, but these errors were encountered: