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

Package integration is broken after Angular CDK 14.1.0 #448

Closed
Sergiobop opened this issue Jul 20, 2022 · 4 comments · Fixed by #449
Closed

Package integration is broken after Angular CDK 14.1.0 #448

Sergiobop opened this issue Jul 20, 2022 · 4 comments · Fixed by #449
Labels

Comments

@Sergiobop
Copy link

Sergiobop commented Jul 20, 2022

After updating CDK to 14.1.0, the integration with cdk-virtual-scroll-viewport is acting weird, adding spaces and scroll when is not neccesary.
Reverting to 14.0.6 fixes the issue.

I think this PR can be related angular/components#24394

Stackblitz:
https://ngx-scrollbar-c3u9al.stackblitz.io

As you can see, i only updated CDK and Material to 14.1.0. In the virtual scroll example, you can see a 'cdk-virtual-scroll-spacer' with height: 50000px (an enormous blank space) after the elements:
Captura de Pantalla 2022-07-20 a las 21 08 25

@MurhafSousli
Copy link
Owner

I think you shared the wrong link, I don't the issue there!

Please check it again, and send me the editable link, not only the preview.

https://stackblitz.com/edit/ngx-scrollbar-c3u9al?file=src%2Fapp%2Fvirtual-scroll-example%2Fvirtual-scroll-example.html

@Sergiobop
Copy link
Author

Sergiobop commented Jul 22, 2022

Oh sorry, in the link you provided, you can already see the problem:
https://stackblitz.com/edit/ngx-scrollbar-c3u9al?file=src%2Fapp%2Fvirtual-scroll-example%2Fvirtual-scroll-example.html

Grabación de pantalla 2022-07-22 a las 9 17 41

@NimeTop
Copy link

NimeTop commented Aug 5, 2022

You got any chance to look into this one @MurhafSousli ? Thanks!

@MurhafSousli
Copy link
Owner

MurhafSousli commented Aug 7, 2022

Fixed in v10.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants