angular-ui/ui-grid

CellNav prevents menu from closing

Open

#5.734 aberto em 11 de out. 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

When cellNav is turned on, it has a stopPropogation on the cell element that prevents the grid/column menu from closing.

To see any example, go to the cellNav tutorial: http://ui-grid.info/docs/#/tutorial/202_cellnav

navigate through some cells, open up a menu, then click away from the menu

Result: the menu remains open Expected Result: the menu would close

This is due to the stopPropogation in the cellNav click event:

https://github.com/angular-ui/ui-grid/blob/master/src/features/cellnav/js/cellnav.js#L1044-L1045

This prevents the menu hide event from occurring:

https://github.com/angular-ui/ui-grid/blob/master/src/js/core/directives/ui-grid-menu.js#L111

What is the need/reason behind the stopPropogation in cellNav?

Guia do colaborador

Direção de pesquisa
Inspecione a chamada stopPropagation nas linhas 1044-1045 do cellnav.js e determine se pode ser removida ou aplicada condicionalmente para permitir a propagação do evento de ocultação do menu.
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScriptAngular
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75