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 sets the wrong height for items containing long word-wrapped text inside a * column #17050

Closed
mfeingol opened this issue Aug 29, 2023 · 2 comments
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView layout-grid 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

Comments

@mfeingol
Copy link

Description

See description and repro steps

Steps to Reproduce

Inside a CollectionView, define the item DataTemplate using a Grid with several columns with Width=Auto and one column with Width=* containing a Button with long text and LineBreakMode set to WordWrap. You will see the incorrect rendering:

image

Using hot reload, add a character to the Button's text and save. You will see the correct rendering:

image

Link to public reproduction project repository

https://github.com/mfeingol/repros/tree/master/CollectionViewHeightBug

Version with bug

8.0.0-preview.7.8842

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

Affected platform versions

Android 13

Did you find any workaround?

Setting a min height works, but is very annoying.

Relevant log output

No response

@mfeingol mfeingol added the t/bug Something isn't working label Aug 29, 2023
@Eilon Eilon added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label Aug 30, 2023
@samhouts samhouts added layout-grid migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert labels Aug 30, 2023
@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Sep 4, 2023
@XamlTest
Copy link
Collaborator

XamlTest commented Sep 4, 2023

Verified this on Visual Studio Enterprise 17.8.0 Preview 1.0. Repro on Android 13.0-API33 .NET 8 with below Project:
CollectionViewHeightBug.zip

Windows:
image

Android:
image

iOS:
image

@mfeingol
Copy link
Author

This appears to be fixed in MAUI 8.0.6.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 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 layout-grid 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

4 participants