Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] CarouselView, widthrequest and heightrequest issue #6301

Closed
juanlao opened this issue May 24, 2019 · 2 comments
Closed

[Bug] CarouselView, widthrequest and heightrequest issue #6301

juanlao opened this issue May 24, 2019 · 2 comments
Labels
a/carouselview a/layout e/6 🕕 6 partner Issue submitted by partner or related open source project t/bug 🐛

Comments

@juanlao
Copy link

juanlao commented May 24, 2019

Description

Having a CarouselView control with a data template containing a grid does not inherit Width and Height values.

Steps to Reproduce

  1. Clone or download the sample I created on https://github.com/juanlao/xamarincarouselview
  2. Run the sample

Expected Behavior

Two carrouselView must be shown with the same images to do a carrouselView.

Actual Behavior

Only the first carrousel view shows the images. That carrousel has a stackpanel on the datatemplate but because the WidthRequest and HeightRequest properties are binded to the carrouselView.
The problem is that I can not define a datatemplate on other xaml to do the binding.

Basic Information

  • Version with issue: 4.0.0.425677

  • IDE: Visual Studio 2019

  • Platform Target Frameworks:

    • Android: 9.0

Reproduction Link

https://github.com/juanlao/xamarincarouselview

@juanlao juanlao added s/unverified New report that has yet to be verified t/bug 🐛 labels May 24, 2019
@pauldipietro pauldipietro added this to New in Triage May 24, 2019
@samhouts
Copy link
Member

samhouts commented Jun 1, 2019

@juanlao CarouselView is still in very early stages. Thanks for the report! We'll be sure to address it as we continue work.

@samhouts samhouts added e/6 🕕 6 and removed s/unverified New report that has yet to be verified labels Jun 1, 2019
@samhouts samhouts moved this from New to Ready For Work in Triage Jun 1, 2019
@samhouts samhouts added this to To do in Other Ready For Work Jun 1, 2019
@samhouts samhouts removed this from Ready For Work in Triage Jun 1, 2019
@samhouts samhouts added the partner Issue submitted by partner or related open source project label Jun 10, 2019
@samhouts samhouts added this to Backlog in CollectionView Jul 25, 2019
@samhouts samhouts added this to Backlog in CarouselView Jul 25, 2019
@samhouts samhouts removed this from Backlog in CollectionView Jul 25, 2019
@jsuarezruiz
Copy link
Contributor

Hi Juanma. Thanks for the repro sample!. I've been doing some tests.
Updating the sample to Xamarin.Forms 4.3 you will see the following result:
issue6301
The issue is fixed. Keep in mind that if you upgrade to the latest available version, 4.3.0.819712-pre2, you will have to change the Carousel Layout. The Layout supported is LinearItemsLayout.

Other Ready For Work automation moved this from To do to Done Sep 20, 2019
CarouselView automation moved this from Backlog to Done Sep 20, 2019
@samhouts samhouts removed this from Done in Other Ready For Work May 6, 2020
@samhouts samhouts removed this from Done in CarouselView May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/carouselview a/layout e/6 🕕 6 partner Issue submitted by partner or related open source project t/bug 🐛
Projects
None yet
Development

No branches or pull requests

3 participants