search

alexrainman / CarouselView

CarouselView control for Xamarin Forms
MIT License
437 stars 177 forks source link

CarouselView not worked with ListView in Xamarin.Forms version above 2.3.1.114 on Android #26

Closed windsql closed 7 years ago

windsql commented 7 years ago

Add a ListView to the CarouselView it worked well in Xamarin.Forms 2.3.1.114,but not worked above it. CarouselView.zip

alexrainman commented 7 years ago

@windsql the issue is not related to CarouselView. There's a bug in Xamarin.Forms core. This:

Platform.SetRenderer(view, Platform.CreateRenderer(view));

Doesn't works anymore and the exception comes from native android ListView.SetAdapter call.

I cannot do anything about it. My advice? downgrade to previuos version of Forms. Basically the last two version of Forms are buggy.

windsql commented 7 years ago

Ok,I found another CarouselView control,it worked well with the new version of Xamarin.Forms , Xamarin.Forms.CarouselView(https://github.com/alexrainman/CarouselView) maybe you will find something in it.

2016-10-12 11:27 GMT+08:00 Alex Rainman notifications@github.com:

@windsql https://github.com/windsql the issue is not related to CarouselView. There's a bug in Xamarin.Forms core. This:

Platform.SetRenderer(view, Platform.CreateRenderer(view));

Doesn't works anymore and the exception comes from native android ListView.SetAdapter call.

I cannot do anything about it. My advice? downgrade to previuos version of Forms. Basically the last two version of Forms are buggy.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/alexrainman/CarouselView/issues/26#issuecomment-253109511, or mute the thread https://github.com/notifications/unsubscribe-auth/AD03Fe1NROnYGNwJ54Z0k9KhnvYSV_inks5qzFOqgaJpZM4KSMdu .

alexrainman commented 7 years ago

@windsql you just send me a link to my own repo :)

windsql commented 7 years ago

Sorry,the correct is https://github.com/xamarin/Xamarin.Forms.CarouselView

2016-10-12 11:42 GMT+08:00 Alex Rainman notifications@github.com:

@windsql https://github.com/windsql you just send me a link to my own repo :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/alexrainman/CarouselView/issues/26#issuecomment-253111270, or mute the thread https://github.com/notifications/unsubscribe-auth/AD03Fek9ZMdSdNl266xMIYcPqeuVRWMrks5qzFc5gaJpZM4KSMdu .

alanspires commented 7 years ago

So is this to say that you don't plan on supporting newer versions of Xam.Forms?

alexrainman commented 7 years ago

Not until they release an stable version. Both latest releases are buggy and don't play good with Xamarin.iOS 10 and Xamarin.Android 7. Also latest couple of releases of Xamarin Studio generates bugs when you build you app.

alexrainman commented 7 years ago

By the way, carousel will work with latest versions of Forms but you cannot use a ListView inside a page. I reported the problem to Xamarin.

alanspires commented 7 years ago

Unfortunately that's exactly what I'm trying to do lol. What exactly is the issue with that and can I get the bugzilla link so i can track it?

alexrainman commented 7 years ago

https://bugzilla.xamarin.com/show_bug.cgi?id=44846

alanspires commented 7 years ago

Doesn't look like they are going to touch that without a repo.

alexrainman commented 7 years ago

I am going to refer them to this plugin repo, easier way to reproduce it.

alexrainman commented 7 years ago

Fixed.

alexrainman commented 7 years ago

If my work is helping you, please help me back: https://xamarinhq.wufoo.com/forms/nominate-a-xamarin-mvp/

alexrainman commented 7 years ago

This is what i have done that is community visible: