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.

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

CDK Virtual Scroll + Router Active Link changes active item when scrolling #18021

Open
alek-aleksandrov opened this issue Dec 20, 2019 · 0 comments
Labels
area: cdk/scrolling P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent

Comments

@alek-aleksandrov
Copy link

alek-aleksandrov commented Dec 20, 2019

Reproduction

https://stackblitz.com/github/empty-god/scroll-test

Steps to reproduce:

  1. Click on any item in the list, and scroll to get new items
  2. You should see the non-selected items highlighted, in no specific order

Expected Behavior

The list should continue scrolling with no further highlight changes without a new click.

Actual Behavior

The list seems to select a similarly placed item instead of referencing the correct variable in selection.

Environment

  • Angular: 8.2.13
  • CDK/Material: 8.2.3
  • Browser(s): Chrome 79.0.3945.79
  • Operating System (e.g. Windows, macOS, Ubuntu): Catalina 10.14.6

Notes

This seems to be fixed when you use templateCacheSize is set to 0.

@mmalerba mmalerba added the needs triage This issue needs to be triaged by the team label May 20, 2020
@mmalerba mmalerba added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent and removed needs triage This issue needs to be triaged by the team labels Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: cdk/scrolling P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
Projects
None yet
Development

No branches or pull requests

4 participants
@mmalerba @alek-aleksandrov @andrewseguin and others