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

[Windows] Resizing window causes CarouselView glitches #17829

Open
rachelkang opened this issue Oct 4, 2023 · 6 comments
Open

[Windows] Resizing window causes CarouselView glitches #17829

rachelkang opened this issue Oct 4, 2023 · 6 comments
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@rachelkang
Copy link
Member

rachelkang commented Oct 4, 2023

Description

Resizing a window on a page with a CarouselView causes the CarouselView sizing, image rendering, and position to glitch inconsistently.

CarouselViewWindowResizeGlitches.mp4

Steps to Reproduce

  1. Run recipeSearch repo on rc2
  2. Deploy to Windows machine
  3. Click on "My Recipes" tab
  4. Resize window
  5. Observe erratic behavior

Link to public reproduction project repository

https://github.com/rachelkang/recipeSearch

Version with bug

8.0.0-rc.1.9171

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

8.0.0-preview.7.8842

Affected platforms

Windows

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response

@rachelkang rachelkang added t/bug Something isn't working potential-regression This issue described a possible regression on a currently supported version., verification pending labels Oct 4, 2023
@rachelkang
Copy link
Member Author

NOT reproducible on net7 (7.0.401)

There's also a spacing difference. In net7, all the items render right next to each other whereas in net8, only one item is shown at a time.

@rachelkang rachelkang added i/regression This issue described a confirmed regression on a currently supported version platform/windows 🪟 area-controls-collectionview CollectionView, CarouselView, IndicatorView and removed potential-regression This issue described a possible regression on a currently supported version., verification pending labels Oct 4, 2023
@samhouts samhouts changed the title [Windows] Resizing window causes CarouselView glitches [regression/8.0.0-rc.1.9171] [Windows] Resizing window causes CarouselView glitches Oct 5, 2023
@samhouts samhouts added this to the .NET 8 GA milestone Oct 5, 2023
@samhouts
Copy link
Member

samhouts commented Oct 5, 2023

Confirmed that this changed between 8.0.0-preview.7.8842 and 8.0.0-rc.1.9171, likely from #16842. However, I'm not sure if it's actually a regression or if there is perhaps code in the sample app that was written to workaround the weird behavior of CarouselView in Windows before the change was made? @rachelkang please investigate! Thanks!

@rachelkang
Copy link
Member Author

Removing this code from the sample does prevent the glitchy behavior from surfacing!

Based on some of the comments in the PR that describe grey blocks on resizing, it does sound like the glitchy behavior might be part of a long-standing known issue: #16842 (comment)
so it might still be worth keeping this open unless it's already being tracked elsewhere

@samhouts
Copy link
Member

samhouts commented Oct 5, 2023

Love it! I'm going to downgrade this from a regression, but we'll keep it open to track the glitchy bug. THANK YOU

@samhouts samhouts removed the i/regression This issue described a confirmed regression on a currently supported version label Oct 5, 2023
@samhouts samhouts changed the title [regression/8.0.0-rc.1.9171] [Windows] Resizing window causes CarouselView glitches [Windows] Resizing window causes CarouselView glitches Oct 5, 2023
@samhouts samhouts modified the milestones: .NET 8 GA, .NET 8 + Servicing Oct 5, 2023
@kevinxufei kevinxufei added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Feb 28, 2024
@kevinxufei
Copy link

This issue was verified in Visual Studio 17.10.0 Preview 1 (maui version: 8.0.6). Use the sample project to reproduce the issue on the Windows platform.

@bronteq
Copy link

bronteq commented May 17, 2024

Probably solved by this #22222

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants