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

CollectionView in Toolkit Popup not work properly #20012

Closed
FrancescoGiancristofaro opened this issue Jan 19, 2024 · 5 comments
Closed

CollectionView in Toolkit Popup not work properly #20012

FrancescoGiancristofaro opened this issue Jan 19, 2024 · 5 comments
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@FrancescoGiancristofaro
Copy link

FrancescoGiancristofaro commented Jan 19, 2024

Description

CollectionView in a Popup not work correctly in these two situation:

  • if property Size of Popup is set
  • if nested into a Frame

Steps to Reproduce

Run attached project and try both Frame popup and Sized popup buttons

Link to public reproduction project repository

https://github.com/FrancescoGiancristofaro/CollectionViewPopupIssue/tree/main

Version with bug

8.0.6

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

Android, I was not able test on other platforms

Affected platform versions

Android 14

Did you find any workaround?

In sized popup case by removing size prop.
In frame popup case by removing frame and use an another control.

Relevant log output

No response

@FrancescoGiancristofaro FrancescoGiancristofaro added the t/bug Something isn't working label Jan 19, 2024
@Eilon Eilon added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label Jan 20, 2024
@ajay-mundi
Copy link

Honestly I just avoid using frames like the plague, I stick with border and things (generally) work well.

@samhouts samhouts added the migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert label Jan 23, 2024
@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Mar 5, 2024
@XamlTest
Copy link

XamlTest commented Mar 5, 2024

Verified this on VS 17.10.0 Preview 1.0(8.0.6). Repro on Android 14.0-API34, not repro on iOS 17.2 with below Project:
CollectionViewPopupIssue.zip

Android:
image

@jsuarezruiz
Copy link
Contributor

@jfversluis Should we move this one to the Toolkit repo?

@jfversluis
Copy link
Member

Yeah seems like this is more a thing in Popup then CollectionView, lets move it there until we prove otherwise.

@jfversluis
Copy link
Member

@FrancescoGiancristofaro I'm very sorry, but due to technical reasons (see linked above) we're not able to move this to the https://github.com/CommunityToolkit/Maui repo automatically. If this is still something that you're experiencing, could you maybe open an issue there as its more relevant to the Toolkit?

Thanks!

@jfversluis jfversluis closed this as not planned Won't fix, can't repro, duplicate, stale Jun 20, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 21, 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 migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/android 🤖 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

7 participants