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

[Bug] [Grid] Row's height gets messed up in X.F 4.8 sr5 #13177

Closed
inimirpaz opened this issue Dec 17, 2020 · 7 comments
Closed

[Bug] [Grid] Row's height gets messed up in X.F 4.8 sr5 #13177

inimirpaz opened this issue Dec 17, 2020 · 7 comments
Assignees
Labels
a/grid a/layout i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often s/unverified New report that has yet to be verified t/bug 🐛
Projects

Comments

@inimirpaz
Copy link

inimirpaz commented Dec 17, 2020

Description

(see attached project)
With absolute no change to the code, since Xamarin.Forms 4.8.0.1687 (4.8 sr5) the height of the rows in a grid is different from previous versions (shorter) and gets completely messed up if you play with the visibility of the last row items of a grid. Both on iOS and Android.

Steps to Reproduce

  1. Have a grid with more than 1 row item
  2. Make the last items visibility change
  3. Switch between 4.8 sr5 and sr4 back and forth

Expected Behavior

Same as before 4.8 sr5, that is, rows should keep their height and most importantly they must not shrink when changing the visibility of row items.

Basic Information

  • Version with issue: 4.8.0.1687
  • Last known good version: 4.8.0.1560 and lower
  • Platform Target Frameworks:
    • iOS: (tested on 14.2)
    • Android: (tested on Android 10 and lower)

Screenshots

4.8 sr4: correct behavior (default row height is ok; when showing/hiding items rows keep their height still)
4_8_0_1560

4.8 sr5: wrong behavior (default row height is shorter when compared with previous version; showing/hiding items get the row item all messed up)
4_8_0_1687

Left 4.8 sr4, right 4.8 sr5
comparison

Reproduction Link

GridRowBug.zip

@inimirpaz inimirpaz added s/unverified New report that has yet to be verified t/bug 🐛 labels Dec 17, 2020
@samhouts samhouts added this to New in Triage Dec 17, 2020
@jsuarezruiz jsuarezruiz added a/grid a/layout i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often labels Dec 17, 2020
@inimirpaz
Copy link
Author

inimirpaz commented Dec 18, 2020

Latest X.F release (4.8.0.1821 sr6) seems to have fixed this issue (?)... Might be a duplicate of #13034

@jsuarezruiz
Copy link
Contributor

This issue seems to be fixed by #12475
cc @hartez

@jsuarezruiz jsuarezruiz moved this from New to Needs Info in Triage Dec 18, 2020
@jsuarezruiz jsuarezruiz added the s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. label Dec 18, 2020
@CometSL9
Copy link

The issue persists on XF 4.8.0.1821 under WPF.

image

The word "missing" comes from the label placed in (*) width sized grid column 4. The grid row height is calculated incorrect.

Very annoying!

@Redth Redth removed the s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. label Jan 21, 2021
@PureWeen
Copy link
Contributor

@CometSL9 can you test with the latest release of 5.0 ?
There were a number of changes made there

@Redth Redth moved this from Needs Info to New in Triage Jan 25, 2021
@CometSL9
Copy link

Sure, it still persists as of v5.0.0.1931.

The bug is really annoying because it hinders this part of my project from getting the "OK" of the colleagues.

Screenshot:
image

Code to reproduce the error:
<ListView HasUnevenRows="True" HeightRequest="200" HorizontalOptions="Center" WidthRequest="300"> <ListView.ItemTemplate> <DataTemplate> <ViewCell> <Grid> <Grid.RowDefinitions> <RowDefinition Height="Auto" /> </Grid.RowDefinitions> <Grid.ColumnDefinitions> <ColumnDefinition Width="100" /> <ColumnDefinition Width="*" /> </Grid.ColumnDefinitions> <Label Grid.Column="0" Text="Fixed width" /> <Label Grid.Column="1" LineBreakMode="WordWrap" Text="{Binding .}" /> </Grid> </ViewCell> </DataTemplate> </ListView.ItemTemplate> <ListView.ItemsSource> <x:Array Type="{x:Type x:String}"> <x:String>Test</x:String> <x:String>Text is too long to display on a single line</x:String> <x:String>Another Test</x:String> </x:Array> </ListView.ItemsSource> </ListView>

@rmarinho
Copy link
Member

@CometSL9 can you please upload a small reproduction? Or open a new issue ? thanks!

@rmarinho
Copy link
Member

Closing at this issue is fixed on 4.8-sr6 , @CometSL9 please open a new issue with your reproduction

Triage automation moved this from New to Closed Jan 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/grid a/layout i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often s/unverified New report that has yet to be verified t/bug 🐛
Projects
Triage
  
Closed
Development

No branches or pull requests

7 participants