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

[Android] Performance issues with CollectionView using grouping and variable sizes #22192

Closed
jsuarezruiz opened this issue May 3, 2024 · 3 comments
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView delighter-sc platform/android 🤖 s/triaged Issue has been reviewed t/bug Something isn't working

Comments

@jsuarezruiz
Copy link
Contributor

Description

Performance issues with CollectionView using grouping and variable sizes.
perf-groups-variable-size

Steps to Reproduce

  1. Sync or download https://github.com/dotnet/maui-samples/tree/main/8.0/UserInterface/Views/CollectionViewDemos
  2. Launch the App on Android.
  3. Navigate to Grouping with variable sized items.

Link to public reproduction project repository

https://github.com/dotnet/maui-samples/tree/main/8.0/UserInterface/Views/CollectionViewDemos

Version with bug

8.0.21 SR4.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

Android 14

Did you find any workaround?

No response

Relevant log output

No response

@jsuarezruiz jsuarezruiz added t/bug Something isn't working platform/android 🤖 legacy-area-perf Startup / Runtime performance area-controls-collectionview CollectionView, CarouselView, IndicatorView labels May 3, 2024
@dotnet-policy-service dotnet-policy-service bot added the s/triaged Issue has been reviewed label May 3, 2024
Copy link
Contributor

github-actions bot commented May 3, 2024

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@jsuarezruiz
Copy link
Contributor Author

jsuarezruiz commented May 3, 2024

I didn't realize, but the issue is because use the following hierarchy:

StackLayout > CollectionView

Changing it by:

Grid > CollectionView

Fixes the performance. However, after the changes and scrolling:
image
image

@jsuarezruiz jsuarezruiz removed the legacy-area-perf Startup / Runtime performance label May 3, 2024
@PureWeen
Copy link
Member

PureWeen commented May 3, 2024

Duplicate of #18834

@PureWeen PureWeen marked this as a duplicate of #18834 May 3, 2024
@PureWeen PureWeen closed this as completed May 3, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jun 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView delighter-sc platform/android 🤖 s/triaged Issue has been reviewed t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants