-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Settings Page Animation Restarts When Switching To Settings Tab #10632
Comments
I wonder if this is because of us doing a |
(Thanks for filing this!) |
I did a trace because I was curious. Looks something like this:
So yeah, looks like XAML is just doing it for us. Playing with the XAML controls gallery, I agree that perhaps a DrillIn or Suppress may work best. Definitely odd! |
@carlos-zamora - do you have a preference between Drill in or Surpress? 😊 |
Let's try DrillIn! |
@carlos-zamora - WDYT? |
OK. Committing :) |
## PR Checklist * [x] Closes #10632 * [x] CLA signed. * [ ] Tests added/passed * [ ] Documentation updated. * [ ] Schema updated. * [ ] I've discussed this with core contributors already.
## PR Checklist * [x] Closes #10632 * [x] CLA signed. * [ ] Tests added/passed * [ ] Documentation updated. * [ ] Schema updated. * [ ] I've discussed this with core contributors already.
🎉This issue was addressed in #10934, which has now been successfully released as Handy links: |
🎉This issue was addressed in #10934, which has now been successfully released as Handy links: |
When you have a setting tab already open and switch back and forth between the Settings tab and any other tab, the animation of the page (where the page content appears vertically centered and moves upward - which is a bit of an odd animation itself) will restart again, even though this isn't the first time opening the page.
It might have been intended, but I find it a little.... odd.
Proposed technical implementation details (optional)
A different animation might make this appear less odd. Like fade, or any number of animations.
2021-07-12.09-30-48.mp4
The text was updated successfully, but these errors were encountered: