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

[iOS] Collectionview itemslayout styling changes with footer #21728

Closed
Uridel opened this issue Apr 9, 2024 · 2 comments · Fixed by #24564
Closed

[iOS] Collectionview itemslayout styling changes with footer #21728

Uridel opened this issue Apr 9, 2024 · 2 comments · Fixed by #24564
Assignees
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView fixed-in-8.0.90 fixed-in-9.0.0-rc.2.24503.2 migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@Uridel
Copy link

Uridel commented Apr 9, 2024

Description

When using a collectionview that has the itemslayout property and a footer the styling becomes all wrong.
So far I have only encountered this on iOS and when there is 1 item in the collection.

Code to reproduce:

`

<ContentPage.Resources>
</ContentPage.Resources>

<CollectionView x:Name="collection" ItemsLayout="VerticalGrid, 2">
	<CollectionView.ItemTemplate>
		<DataTemplate>
			<Label Text="{Binding Name}" BackgroundColor="red" HeightRequest="200" HorizontalOptions="FillAndExpand" VerticalOptions="FillAndExpand"/>
		</DataTemplate>
	</CollectionView.ItemTemplate>
	<CollectionView.Footer>
		<ContentView >
			<ContentView.ControlTemplate>
				<ControlTemplate>
					<Grid RowDefinitions="Auto, *">
						<ContentPresenter Grid.Row="0"/>
						<ContentView Grid.Row="1"  HeightRequest="100" InputTransparent="True"/>
					</Grid>
				</ControlTemplate>
			</ContentView.ControlTemplate>
		</ContentView>
	</CollectionView.Footer>
</CollectionView>
`

Steps to Reproduce

  1. Create a collectionview with 1 item and add ItemsLayout="VerticalGrid, 2"
  2. Add a Footer with a contentview controltemplate
  3. have it contain any 2 controls
  4. The item will be displayed in the middle instead of to the left.

Expected behavior
The item is shown on the left side of the page

Actual behavior
The item is shown on the middle of the page

Link to public reproduction project repository

https://github.com/Uridel/CollectionviewLayout

Version with bug

8.0.14 SR3.1

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

No response

Did you find any workaround?

no

Relevant log output

No response

@Uridel Uridel added the t/bug Something isn't working label Apr 9, 2024
@PureWeen PureWeen added area-controls-collectionview CollectionView, CarouselView, IndicatorView s/needs-repro Attach a solution or code which reproduces the issue labels Apr 16, 2024
@dotnet-policy-service dotnet-policy-service bot added the s/no-recent-activity Issue has had no recent activity label Apr 22, 2024
@samhouts samhouts added the migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert label Apr 22, 2024
@dotnet-policy-service dotnet-policy-service bot added s/no-recent-activity Issue has had no recent activity and removed s/no-recent-activity Issue has had no recent activity labels Apr 22, 2024
@Uridel
Copy link
Author

Uridel commented May 1, 2024

Here is a link to the project with which it can be reproduced https://github.com/Uridel/CollectionviewLayout

@dotnet-policy-service dotnet-policy-service bot added s/needs-attention Issue has more information and needs another look and removed s/needs-repro Attach a solution or code which reproduces the issue s/no-recent-activity Issue has had no recent activity labels May 1, 2024
@QianaJiao
Copy link

Can repro this issue at iOS platform on the latest 17.10 Preview 6 (8.0.3 & 8.0.21).

@QianaJiao QianaJiao added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed potential-regression This issue described a possible regression on a currently supported version., verification pending and removed potential-regression This issue described a possible regression on a currently supported version., verification pending labels May 6, 2024
@jsuarezruiz jsuarezruiz removed the s/needs-attention Issue has more information and needs another look label May 20, 2024
@jsuarezruiz jsuarezruiz added this to the Backlog milestone May 20, 2024
@samhouts samhouts removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 3, 2024
@samhouts samhouts added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 10, 2024
@mikeparker104 mikeparker104 added the partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with label Jul 25, 2024
@PureWeen PureWeen removed this from the Backlog milestone Aug 8, 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 fixed-in-8.0.90 fixed-in-9.0.0-rc.2.24503.2 migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Status: Done
7 participants