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

[Bug][iOS] CollectionView footer causes focused row to scroll off screen when the Keyboard opens #9706

Open
lobbo232 opened this issue Feb 25, 2020 · 5 comments
Labels
a/collectionview e/4 🕓 4 p/iOS 🍎 t/bug 🐛 up-for-grabs We welcome community contributions to any issue, but these might be a good place to start!

Comments

@lobbo232
Copy link

Description

Found whilst looking for a workaround for issue #9691.

If you have a footer at the bottom of a CollectionView, when clicking on an entry within the CollectionView and the keyboard opens the UI scrolls the additional height of the footer through the CollectionView and moves the tapped entry off the screen.

Steps to Reproduce

  1. Create a new Xamarin.Forms app
  2. Create a CollectionView with a bound collection of at least 20 items with the DataTemplate containing an Entry.
  3. Add a CollectionView.Footer of a StackLayout with of a HeightRequest="1000"
  4. Run the app and tap any of Entry that isn't near the bottom (first Entry works fine).
  5. The focused Entry will be moved off the top of the screen.

Expected Behavior

The focused entry should be focused above the keyboard so the user can see what they type.

Actual Behavior

The focused entry is scrolled off the top of the screen.

Basic Information

  • Version with issue: 4.4.0.991640
  • Last known good version: Unknown
  • IDE: Visual Studio 2019
  • Platform Target Frameworks:
    • iOS: 13.10.0.17
    • Android: 10.1.3.7
    • UWP:
  • Android Support Library Version:
  • Nuget Packages:
  • Affected Devices: iOS

Screenshots

IMG_0206
TAP ITEM 1
IMG_0207

Reproduction Link

Test Footer Causes Elements Out Of Focus.zip

Workaround

@lobbo232 lobbo232 added s/unverified New report that has yet to be verified t/bug 🐛 labels Feb 25, 2020
@pauldipietro pauldipietro added this to New in Triage Feb 25, 2020
@lobbo232 lobbo232 changed the title [Bug][iOS] CollectionView footer height brakes Entry control focus when the Keyboard opens [Bug][iOS] CollectionView footer causes focused row to scroll off screen when the Keyboard opens Feb 25, 2020
@lobbo232
Copy link
Author

After a bit more playing around with this, it seems that with a HieghtRequest of 1000 on the footer content, an iPhone X running iOS 13 is fine, however an iPhone 6 running iOS 12 is not. It might be because the height of the footer is more noticeable on the smaller screen of an iPhone 6

@jsuarezruiz jsuarezruiz added a/collectionview p/iOS 🍎 and removed s/unverified New report that has yet to be verified labels Feb 25, 2020
@jsuarezruiz jsuarezruiz moved this from New to Needs Estimate in Triage Feb 25, 2020
@jsuarezruiz
Copy link
Contributor

@lobbo232 I am going to close #9691 because it is the same issue, ok?

@lobbo232
Copy link
Author

lobbo232 commented Feb 25, 2020

@jsuarezruiz While they are both related to CollectionView scrolling, I don't think they are the same issue.
#9691 is an issue caused by the keyboard obstructing items in a CollectionView.
#9706 is an issue caused by the height of a footer not being taken in to account when focusing an item in a CollectionView.

If you've confirmed both and are happy they're caused by the same thing, then go ahead.

@jsuarezruiz
Copy link
Contributor

I think the cause could be the same, but I will relate and keep them until be 100% sure.

@samhouts samhouts added this to Backlog in CollectionView Mar 10, 2020
@samhouts samhouts added the e/4 🕓 4 label Mar 20, 2020
@samhouts samhouts moved this from Needs Estimate to Ready For Work in Triage Mar 20, 2020
@samhouts samhouts added this to To do in iOS Ready For Work Mar 20, 2020
@samhouts samhouts removed this from Ready For Work in Triage Mar 20, 2020
@samhouts samhouts added this to the 5.0.0 milestone Aug 13, 2020
@samhouts samhouts added this to To do in vNext+1 (5.0.0) Aug 14, 2020
@samhouts samhouts added inactive Issue is older than 6 months and needs to be retested help wanted We welcome community contributions to any issue, but these might be a good place to start! up-for-grabs We welcome community contributions to any issue, but these might be a good place to start! and removed inactive Issue is older than 6 months and needs to be retested help wanted We welcome community contributions to any issue, but these might be a good place to start! labels Sep 18, 2020
@samhouts samhouts removed this from the 5.0.0 milestone Nov 2, 2020
@rhodesRowan
Copy link

why has this been removed? This is a really urgent issue for many people

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/collectionview e/4 🕓 4 p/iOS 🍎 t/bug 🐛 up-for-grabs We welcome community contributions to any issue, but these might be a good place to start!
Projects
CollectionView
  
Review Backlog
Development

No branches or pull requests

4 participants