Bug 53834 - incorrect row heights on ios when using groupheadertemplate in Xamarin.Forms
Summary: incorrect row heights on ios when using groupheadertemplate in Xamarin.Forms ...
Alias: None
Product: Forms
Classification: Xamarin
Component: iOS (show other bugs)
Version: 2.3.4
Hardware: PC Windows
: --- normal
Target Milestone: ---
Assignee: Jimmy [MSFT]
URL: https://github.com/gentledepp/Xamarin...
Depends on:
Reported: 2017-03-23 09:04 UTC by Alexander Marek
Modified: 2017-03-24 16:53 UTC (History)
5 users (show)

See Also:
Tags: bug, ios, rowheight, listview, groupheadertemplate
Is this bug a regression?: Yes
Last known good build:

repro project (67.02 KB, application/x-zip-compressed)
2017-03-23 16:24 UTC, Jimmy [MSFT]

Description Alexander Marek 2017-03-23 09:04:19 UTC

Pre-5 fixes the issue with the invalid row heights in iOS, when there list item has context actions.
However, it destroys row heights in case you have a grouped listview with a custom header template.

To reproduce this issue, I forked 

where he wrote about grouped listviews

What I did then was:
1) I upgraded to Xamarin.Forms
2) I added a custom groupitemtempalte in the MonkeysPage.xaml:

                <!-- having a custom header tempalte leads to incorrect cell rowheights on iOS -->
                        <ViewCell Height="25">
                            <StackLayout VerticalOptions="FillAndExpand"
                                <Label Text="{Binding Key}" TextColor="DimGray" VerticalOptions="Center"/>

You find the corresponding branch here:

Please fix.
Comment 1 Jimmy [MSFT] 2017-03-23 15:03:57 UTC
Thank you for filing this report! Let me run the project you provided to see if I can confirm this issue and I will update the report accordingly.
Comment 2 Jimmy [MSFT] 2017-03-23 16:24:03 UTC
Created attachment 20766 [details]
repro project

I was able to reproduce this issue with sample provided. I am attaching a slightly minimized version of the project to the report.

This issue is similar to bug 51536, the difference is that this bug is happening after adding a GroupHeaderTemplate to the ListView, and it is still occurring with 2.3.4-pre5.

After testing with different Forms versions, this appears to be a regression beginning in the 2.3.4 pre-releases.

### Steps to reproduce
1. Run the attached iOS project

### Expected Results 
The row heights will fit the cell contents

### Actual Results
The row heights are incorrect similar to bug 51536.

### Regression Test    BAD
2.3.4-pre5          BAD
2.3.4-pre3          BAD           GOOD
Comment 3 Samantha Houts [MSFT] 2017-03-24 01:40:17 UTC
Comment 4 Rui Marinho 2017-03-24 01:50:45 UTC
Should be fixed on 2.3.4-pre6
Comment 5 Alexander Marek 2017-03-24 09:17:05 UTC
Did you make sure that your fix does not re-introduce this bug?


see PR here: https://github.com/xamarin/Xamarin.Forms/pull/787
Comment 6 Samantha Houts [MSFT] 2017-03-24 16:53:47 UTC
Yes. This issue was entirely unrelated to Bug 51536. Thank you!

Note You need to log in before you can comment on or make changes to this bug.