-
Notifications
You must be signed in to change notification settings - Fork 7
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
[ABW-3654] - Update screens navigation transitions #1120
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work. My only notes here are for these 2 screens
- EULA, does this need to be a slide up? I guess the reason it is a slide up is because it is an one-time popup?
- Account recovery in troubleshooting screen appears as a slide up, when the button displays an arrow to the right. Could you take a look?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only minor thing I noticed is that when you go from persona edit back to persona details, persona details screen seem to blink/fade in instead of stay in place when persona edit slide down. But looking at the code it seem your setup is correct
Good catch. Fixed it. |
Jira Ticket
Description
This PR updates navigation transitions between screens.