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

#696 Fix not all items visible in MatVirtualScroll #722

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexandreberton
Copy link

Fix wrong formula to allow to displayed all items when scrollviewer is visible

@enkodellc
Copy link
Collaborator

@alexandreberton can you tell me the steps to recreate the issue? Thanks for contributing

@SamProf
Copy link
Owner

SamProf commented Nov 1, 2020

@alexandreberton Can you please explain issue?
Maybe it's better to give example in http://blazorfiddle.com

@alexandreberton
Copy link
Author

Hello. Sorry but I'm unable to give an example with http://blazorfiddle.com because matblazor part does not compile when I tryed.

What I did is pretty simple.
Have a page with the MatVirtualScroll with heigh set to 100%.
In this define itemtemplate with specific heigh (75px in my case).
Have a lot of items in the list.
When you scroll you will not be able to see many items.
I just fixed the formula because in the code you move the scrollbar to the right position but you also decrease the size of the control which is not good.
Hope it helps.

@Kylar182
Copy link

Kylar182 commented May 8, 2021

I have the same issue, looking at the code he changed this an obvious bug and obvious bugfix. It shouldn't require a fiddle example.

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

Successfully merging this pull request may close these issues.

4 participants