angular-ui/ui-grid

Scrolling editable rows with down arrow triggers change event with wrong cell values

Open

#6 298 ouverte le 6 juil. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
good first issue

Métriques du dépôt

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

Description

The example below is originally from the tutorials. Browser is Chrome.

  1. plunkr
  2. select any name
  3. Edit mode is on
  4. now press and hold the down arrow until the grid starts scrolling.
  5. you will now see the details of the last cell changed in the 'last cell edited' field. Although i have not changed anything.

The change should trigger only when the old/new values of the cell has changed. I am simply skipping the 'afterCellEdit' event when the new/old values are the same.

Guide contributeur

Direction de recherche
Examinez le gestionnaire d'événements afterCellEdit dans ui grid pour garantir que les modifications ne soient déclenchées que lorsque les valeurs des cellules changent réellement. Le rapporteur suggère de sauter l'événement lorsque les anciennes et nouvelles valeurs sont identiques. Le problème inclut un exemple plunkr (http://plnkr.co/edit/WizlYTzwdif3Exp1pTDT?p=preview) pour reproduire le bug. Examinez la fonctionnalité d'édition de cellule de la grille et la propagation des événements pour implémenter la correction.
Stack technique
javascript
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65