Skip to content
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.

Sign up for GitHub

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

Jump to bottom

unexpected transition when router change #2697

Closed
vthinkxie opened this issue Dec 27, 2018 · 7 comments · Fixed by #2895
Closed

unexpected transition when router change #2697

vthinkxie opened this issue Dec 27, 2018 · 7 comments · Fixed by #2895
Assignees
@vthinkxie
Labels
🤚 Blocked By @angular/cdk 🤚 Blocked By @angular 🐛 Bug

Comments

@vthinkxie
Copy link
Member

https://stackblitz.com/edit/ng-zorro-antd-start-qgr95a?file=src%2Fapp%2Fhome%2Fhome.component.html
@blackraindrop
Copy link
Contributor

https://stackblitz.com/edit/ng-zorro-antd-start-sbvjt5

@blackraindrop
Copy link
Contributor

https://stackblitz.com/edit/ng-zorro-antd-start-nqwret
把导航从路由组件移动到app.component不会闪烁。

@blackraindrop
Copy link
Contributor

https://stackblitz.com/edit/ng-zorro-antd-start-sxbud9
使用router.navigate代替routerlink也不会闪烁

@vthinkxie
Copy link
Member Author

transition error without ng-zorro-antd
https://stackblitz.com/edit/ng-zorro-antd-start-uvffqm?file=src%2Fapp%2Fapp.component.ts

@vthinkxie
Copy link
Member Author

ref angular/components#14646

@vthinkxie vthinkxie mentioned this issue Dec 27, 2018
Closed
@trotyl
Copy link
Contributor

trotyl commented Dec 30, 2018 •
edited
Loading

For workaround, using imperative className assignment rather than class binding:

https://stackblitz.com/edit/ng-zorro-antd-start-jw8faq?file=src/app/app.component.ts

This is likely to be a Chrome issue triggered by specific timing, while not sure what the real cause, using a different action sequence may get rid of this issue.

@vthinkxie vthinkxie self-assigned this Feb 14, 2019
@vthinkxie vthinkxie added the P0 label Feb 14, 2019
@vthinkxie vthinkxie mentioned this issue Feb 15, 2019
Closed
@vthinkxie vthinkxie removed the P0 label Feb 15, 2019
@vthinkxie vthinkxie mentioned this issue Feb 15, 2019
Merged
3 tasks
@vthinkxie
Copy link
Member Author

ref angular/angular#34718

@vthinkxie vthinkxie reopened this Jan 10, 2020
Ricbet pushed a commit to Ricbet/ng-zorro-antd that referenced this issue Apr 9, 2020
@Ricbet
fix(module:button): fix button animation bug caused by angular
2bf3216 
close NG-ZORRO#2697
@hsuanxyz hsuanxyz mentioned this issue Apr 15, 2020
Closed
hsuanxyz pushed a commit to hsuanxyz/ng-zorro-antd that referenced this issue Aug 5, 2020
fix(module:button): fix button animation bug caused by angular
faca277 
close NG-ZORRO#2697
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vthinkxie vthinkxie

Labels
🤚 Blocked By @angular/cdk 🤚 Blocked By @angular 🐛 Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(module:all): move hostbinding to constructor to fix angular trans…
3 participants
@vthinkxie @blackraindrop @trotyl