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

[Bug] CollectionView with Grouping and GridItemLayout displays wrong number of columns on IOS #13347

Closed
aperel66 opened this issue Jan 8, 2021 · 9 comments · Fixed by #13380
Closed

[Bug] CollectionView with Grouping and GridItemLayout displays wrong number of columns on IOS #13347

aperel66 opened this issue Jan 8, 2021 · 9 comments · Fixed by #13380
Assignees
@hartez
Labels
a/collectionview e/5 🕔 5 i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often in-progress This issue has an associated pull request that may resolve it! t/bug 🐛

Comments

@aperel66
Copy link

aperel66 commented Jan 8, 2021

Description

If you use a CollectionView with IsGrouped = true and ItemsLayout=GridItemsLayout the number of columns display on the IOS is incorrect. Also, the list doesn't scroll all the way down, it seems to be stuck somewhere in the middle

Steps to Reproduce

  1. Use CollectionView with GridItemsLayout 3 columns, ItemsLayoutOrientation.Vertical
  2. Create DataTemplate for Grouping and assign to GroupHeaderTemplate
  3. Populate ItemsSource with GroupedItems as per Microsoft docs https://docs.microsoft.com/en-us/xamarin/xamarin-forms/user-interface/collectionview/grouping
  4. This works perfectly on Android, but not on the IOS - see attached screenshots

Expected Behavior

3 columns are displayed separated by group header

Actual Behavior

only 2 columns on IOS are displayed and the list does not scroll past first page

Basic Information

  • Version with issue:
  • Last known good version:
  • Platform Target Frameworks:
    • iOS:
    • Android:
    • UWP:
  • Android Support Library / AndroidX Version:
  • NuGet Packages:
  • Affected Devices:

Environment

Microsoft Visual Studio Community 2019
Version 16.8.3
VisualStudio.16.Release/16.8.3+30804.86
Microsoft .NET Framework
Version 4.8.04084

Installed Version: Community

ADL Tools Service Provider 1.0
This package contains services used by Data Lake tools

ASA Service Provider 1.0

ASP.NET and Web Tools 2019 16.8.554.20160
ASP.NET and Web Tools 2019

ASP.NET Core Razor Language Services 16.1.0.2052803+84e121f1403378489b842e1797df2f3f5a49ac3c
Provides languages services for ASP.NET Core Razor.

ASP.NET Web Frameworks and Tools 2019 16.8.554.20160
For additional information, visit https://www.asp.net/

Azure App Service Tools v3.0.0 16.8.554.20160
Azure App Service Tools v3.0.0

Azure Data Lake Node 1.0
This package contains the Data Lake integration nodes for Server Explorer.

Azure Data Lake Tools for Visual Studio 2.6.1000.0
Microsoft Azure Data Lake Tools for Visual Studio

Azure Functions and Web Jobs Tools 16.8.554.20160
Azure Functions and Web Jobs Tools

Azure Stream Analytics Tools for Visual Studio 2.6.1000.0
Microsoft Azure Stream Analytics Tools for Visual Studio

C# Tools 3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Common Azure Tools 1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

Extensibility Message Bus 1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.

Fabric.DiagnosticEvents 1.0
Fabric Diagnostic Events

IntelliCode Extension 1.0
IntelliCode Visual Studio Extension Detailed Info

Microsoft Azure HDInsight Azure Node 2.6.1000.0
HDInsight Node under Azure Node

Microsoft Azure Hive Query Language Service 2.6.1000.0
Language service for Hive query

Microsoft Azure Service Fabric Tools for Visual Studio 16.0
Microsoft Azure Service Fabric Tools for Visual Studio

Microsoft Azure Stream Analytics Language Service 2.6.1000.0
Language service for Azure Stream Analytics

Microsoft Azure Stream Analytics Node 1.0
Azure Stream Analytics Node under Azure Node

Microsoft Azure Tools 2.9
Microsoft Azure Tools for Microsoft Visual Studio 2019 - v2.9.30924.1

Microsoft Continuous Delivery Tools for Visual Studio 0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.

Microsoft JVM Debugger 1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

Microsoft Library Manager 2.1.113+g422d40002e.RR
Install client-side libraries easily to any web project

Microsoft MI-Based Debugger 1.0
Provides support for connecting Visual Studio to MI compatible debuggers

Microsoft Visual Studio Tools for Containers 1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.

Mono Debugging for Visual Studio 16.8.43 (00471f8)
Support for debugging Mono processes with Visual Studio.

NuGet Package Manager 5.8.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/

ProjectServicesPackage Extension 1.0
ProjectServicesPackage Visual Studio Extension Detailed Info

SQL Server Data Tools 16.0.62010.06180
Microsoft SQL Server Data Tools

ToolWindowHostedEditor 1.0
Hosting json editor into a tool window

TypeScript Tools 16.0.21016.2001
TypeScript Tools for Microsoft Visual Studio

Visual Basic Tools 3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Visual F# Tools 16.8.0-beta.20507.4+da6be68280c89131cdba2045525b80890401defd
Microsoft Visual F# Tools

Visual Studio Code Debug Adapter Host Package 1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio

Visual Studio Container Tools Extensions 1.0
View, manage, and diagnose containers within Visual Studio.

Visual Studio Tools for Containers 1.0
Visual Studio Tools for Containers

Visual Studio Tools for Kubernetes 1.0
Visual Studio Tools for Kubernetes

VisualStudio.DeviceLog 1.0
Information about my package

VisualStudio.Foo 1.0
Information about my package

VisualStudio.Mac 1.0
Mac Extension for Visual Studio

Xamarin 16.8.000.260 (d16-8@fd405a2)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.

Xamarin Designer 16.8.0.507 (remotes/origin/d16-8@e87b24884)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.

Xamarin Templates 16.8.112 (86385a3)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.

Xamarin.Android SDK 11.1.0.17 (d16-8/c0e2b8e)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: be2226b
Java.Interop: xamarin/java.interop/d16-8@79d9533
ProGuard: Guardsquare/proguard@ebe9000
SQLite: xamarin/sqlite@1a3276b
Xamarin.Android Tools: xamarin/xamarin-android-tools/d16-8@2fb1cbc

Xamarin.iOS and Xamarin.Mac SDK 14.6.0.15 (87a1b18d8)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.

Show/Hide Visual Studio info

Build Logs

Screenshots

IOS ScreenShot - CollectionViewBug
Android CollectionView Correct Behavior

Reproduction Link

Workaround
@aperel66 aperel66 added s/unverified New report that has yet to be verified t/bug 🐛 labels Jan 8, 2021
@hartez hartez added a/collectionview and removed s/unverified New report that has yet to be verified labels Jan 9, 2021
@hartez hartez added e/5 🕔 5 i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often labels Jan 9, 2021
@hartez
Copy link
Contributor

hartez commented Jan 9, 2021

@aperel66 Do you have any horiztonal item spacing set in your layout?

@hartez hartez self-assigned this Jan 9, 2021
@aperel66
Copy link
Author

aperel66 commented Jan 9, 2021

Yes, I have Horizontal spacing set to 5 and Vertical spacing set to 5.

@aperel66
Copy link
Author

aperel66 commented Jan 9, 2021

@hartez Yes, I have Horizontal spacing set to 5 and Vertical spacing set to 5.

@aperel66
Copy link
Author

aperel66 commented Jan 9, 2021

@hartez
This is the same view, with IsGrouped = false;
Simulator Screen Shot - iPhone 12 - 2021-01-09 at 14 45 47

@hartez
Copy link
Contributor

hartez commented Jan 11, 2021

@aperel66 Which version of the Forms NuGet package are you using?

@aperel66
Copy link
Author

aperel66 commented Jan 11, 2021 via email

hartez added a commit that referenced this issue Jan 12, 2021
@hartez
Remove unnecessary edge inset for vertically scrolling groups; Fixes #…
17ee363 
…13347
@hartez hartez mentioned this issue Jan 12, 2021
Merged
2 tasks
@hartez hartez added the in-progress This issue has an associated pull request that may resolve it! label Jan 12, 2021
@aperel66
Copy link
Author

@hartez @rmarinho
Gentlemen, can you please provide an estimate on when this issue will be fixed/released.
Thank you!

@aperel66
Copy link
Author

@hartez @rmarinho Is there a workaround for this issue meanwhile?

@aperel66 aperel66 mentioned this issue Jan 26, 2021
Closed
@hartez
Copy link
Contributor

hartez commented Feb 1, 2021

@aperel66 PR #13380 should fix this issue; we just need to get it through review and testing. Hopefully it will hit in time to make the next service release.

@hartez hartez linked a pull request Feb 2, 2021 that will close this issue
Remove unnecessary edge inset for vertically scrolling groups; Fixes … #13380
Merged
2 tasks
hartez added a commit that referenced this issue Feb 3, 2021
@hartez
Remove unnecessary edge inset for vertically scrolling groups; Fixes #…
3610d56 
…13347
hartez added a commit that referenced this issue Feb 5, 2021
@hartez
Remove unnecessary edge inset for vertically scrolling groups; Fixes #…
9fec16e 
…13347
rmarinho pushed a commit that referenced this issue Feb 9, 2021
@hartez
Remove unnecessary edge inset for vertically scrolling groups; Fixes … (
7ea5159 
#13380) fixes #13347

* Remove unnecessary edge inset for vertically scrolling groups; Fixes #13347

* Fix edge insets in horizontally-scrolling groups
@pablolop002 pablolop002 mentioned this issue Feb 13, 2021
Open
@dependabot dependabot bot mentioned this issue Mar 18, 2021
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@hartez hartez

Labels
a/collectionview e/5 🕔 5 i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove unnecessary edge inset for vertically scrolling groups; Fixes … xamarin/Xamarin.Forms
2 participants
@hartez @aperel66