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
IndicatorLayout isn't being correctly cleared out when bindable property is set to null #22361
Labels
area-controls-collectionview
CollectionView, CarouselView, IndicatorView
good first issue
Good for newcomers
Milestone
Comments
Eilon
added
the
area-controls-collectionview
CollectionView, CarouselView, IndicatorView
label
May 13, 2024
I don't think the code is meaningless, I think the if statement is wrong |
PureWeen
changed the title
Meaningless code found inside the indicatorView.cs file
IndicatorLayout isn't being correctly cleared out when bindable property is set to null
May 13, 2024
kubaflo
added a commit
to kubaflo/maui
that referenced
this issue
May 13, 2024
kubaflo
added a commit
to kubaflo/maui
that referenced
this issue
May 13, 2024
kubaflo
added a commit
to kubaflo/maui
that referenced
this issue
May 13, 2024
This has been here since Xamarin.Forms, unchanged, for five years. I guess no one ever hit it? lol. |
It's really confusing. |
That's true. I've been confused for a long time. |
PureWeen
pushed a commit
that referenced
this issue
May 14, 2024
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
good first issue
Good for newcomers
maui/src/Controls/src/Core/IndicatorView/IndicatorView.cs
Lines 155 to 159 in 39f90ee
Meaningless code found inside the indicatorView.cs file
IndicatorView.IndicatorLayout is already null, still use as to convert to IndicatorStackLayout
The text was updated successfully, but these errors were encountered: