xamarin/Xamarin.Forms

iOS: TableView Selects Incorrect Row when Keyboard is Open

Open

#3,589 opened on Aug 16, 2018

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)C# (1,926 forks)batch import
e/6 :clock6:help wantedin-progressinactivep/iOS 🍎t/bug :bug:up-for-grabs

Repository metrics

Stars
 (5,644 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Description

TableViews in iOS will select the incorrect row if the Keyboard is open for an EntryCell.

Steps to Reproduce

  1. Create a TableView with enough rows to force the screen to scroll
  2. Place an EntryCell as they very last Row
  3. Focus on the EntryCell to open they keypad
  4. Select the Row above the entry cell

Expected Behavior

The cell above the EntryCell will receive a tap event

Actual Behavior

The cell several rows up will actually receive the Tap event

Basic Information

  • Version with issue: 3.1.0 R2
  • Last known good version: none
  • IDE: Visual Studio for Mac
  • Platform Target Frameworks:
    • iOS: 11.4
    • Android: N/A
    • UWP: N/A
  • Android Support Library Version: N/A
  • Nuget Packages: N/A
  • Affected Devices: all iOS

Reproduction Link

https://github.com/jnosek/xf-tableview-test

Contributor guide

Research direction
Examine the TableView's touch handling code when keyboard is active; focus on how view coordinates are calculated during animation.
Tech stack
csharp
Domain
mobile
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Git
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60