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

ListView issue with ScrollViewer/ItemsPanelTemplate #3620

Closed
imadenis opened this issue Jul 1, 2024 · 2 comments · Fixed by #3622
Closed

ListView issue with ScrollViewer/ItemsPanelTemplate #3620

imadenis opened this issue Jul 1, 2024 · 2 comments · Fixed by #3622
Labels
bug evaluation required Items is pending review or evaluation by the team

Comments

@imadenis
Copy link

imadenis commented Jul 1, 2024

Bug explanation

Ticket #3579 ( #3579 ) introduced a breaking behavior in my application.

Having a ListView with a Style that targets the ItemsPanel, the ScrollViewer is not visible anymore at the bottom.

I have made a repo to demonstrate the bug:

https://github.com/imadenis/MaterialDesignBug

I have pinpointed the issue to come from the <Style.Trigger> introduced at line 159
MaterialDesignThemes.Wpf/Themes/MaterialDesignTheme.ScrollViewer.xaml

Version

5.1.0

@imadenis imadenis added bug evaluation required Items is pending review or evaluation by the team labels Jul 1, 2024
@nicolaihenriksen
Copy link
Contributor

@JorisCleVR I am tagging you here because at first glance it seems this was introduced by the padding PR you submitted. Would you have time to look at the issue? If not, no worries, we'll get around to it eventually.

@JorisCleVR
Copy link
Contributor

@nicolaihenriksen It seems that during my PR the IgnorePadding property of the ScrollViewerAssist accidently changed to false by default. I created a PR to fix this: #3622

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug evaluation required Items is pending review or evaluation by the team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants