angular-ui/ui-grid

Async row removal does not work with inline edit

Open

#5.649 aberto em 26 de ago. de 2016

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
help wanted

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

We remove large amounts of data by a background worker and we reflect this on ui-grid. So rows might be removed without any user action (just the refresher which runs in background notices that row is no longer present in database, it is also removed from grid immediately).

The problem is that if user is inline editing any row BELOW such removed row, inline edit still keeps old index.

I have created a plunkr to demonstrate the issue: http://plnkr.co/edit/I8KnXLCxeDiKQq5OR9lC?p=preview

Steps to reproduce:

  1. Start inline edit of second row immediately (e.g. change name and remain in inline edit mode)
  2. After 10 seconds first row is automatically removed
  3. Observe that row with the same index is in edit mode, but actually edited row is moved up. So user is editing another row.

Guia do colaborador

Direção de pesquisa
Reproduza o problema usando o Plunkr fornecido, depois rastreie o tratamento do índice de edição inline quando as linhas são removidas de forma assíncrona.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptAngular
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75