angular-ui/ui-grid

CellNav prevents menu from closing

Open

#5734 aperta il 11 ott 2016

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2496 fork)batch import
help wanted

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Esamina la chiamata stopPropagation nelle righe 1044-1045 di cellnav.js e determina se può essere rimossa o applicata condizionalmente per consentire la propagazione dell'evento di chiusura del menu.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScriptAngular
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75