angular-ui/ui-grid

Small rowEditWaitInterval causes unwanted save with rowedit

Open

#5 224 ouverte le 14 mars 2016

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
help wantedneeds: analysis

Métriques du dépôt

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

Description

It is important to set rowEditWaitInterval to 1 (the lowest available value).

Steps to reporoduce:

  1. Open plunker: http://plnkr.co/edit/ci2rADfto0N4xwdEggAQ?p=preview
  2. Try to edit any cell and navigate to cell on the right or left (don't leave the row). -- If you use arrow keys to navigate everything works fine - save does not occur. -- But if you use mouse (click to another editable cell in the given row) - save should not occur until you leave row, But it does.

It seems that interval is too low to determine that another cell in the row is being edited and grid assumes row was left and attempts to save.

Happens on IE 11 and Edge 13.

Guide contributeur

Direction de recherche
Examinez comment rowEditWaitInterval est utilisé dans la logique d'édition de lignes, en vous concentrant sur la différence entre la navigation au clavier et à la souris lors du déplacement entre les cellules. Vérifiez les gestionnaires d'événements pour la navigation dans les cellules (touches fléchées vs clic) et le mécanisme de déclenchement de la sauvegarde.
Stack technique
javascriptangular
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
GitAngular
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60