xamarin/Xamarin.Forms

HasUnevenRows fires ItemAppearing for invisible screen items when the cell is an Image

Ouverte

#3 874 ouverte le 22 sept. 2018

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C# (1 926 forks)batch import
a/listviewe/5 :clock5:help wantedinactivep/Androidt/bug :bug:up-for-grabs

Métriques du dépôt

Stars
 (5 644 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Description

If a ListView has cells that are made up of only Image instances (or wrapper layouts whose only content is an Image) and HasUnevenRows is set to true, then ItemAppearing will be fired for virtually every list item even if it does not appear on the screen. This is bad for scenarios involving infinite scrolling. The issue seems to exist on both Android and iOS.

Ref: an old issue (27845) that's been resolved.

Steps to Reproduce

  1. Run the attached repro
  2. Observe that ItemAppearing is fired for dozens of items even though it should be fired only a few times upon page load

Reproduction Link

App2.zip

Guide contributeur

Direction de recherche
Reproduisez le probleme en executant l'application jointe. Examinez la logique de mesure et de disposition de ListView avec HasUnevenRows, en particulier lorsque les cellules ne contiennent que des images. Cherchez ou ItemAppearing est declenche et pourquoi il se declenche pour des elements hors ecran.
Stack technique
csharp
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
csharp
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70