angular-ui/ui-grid

CellNav prevents menu from closing

Open

#5.734 geöffnet am 11. Okt. 2016

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
help wanted

Repository-Metriken

Stars
 (5.395 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Überprüfe den stopPropagation Aufruf in cellnav.js Zeile 1044-1045 und stelle fest, ob er entfernt oder bedingt angewendet werden kann, um die Weitergabe des Menü Ausblendereignisses zu ermöglichen.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScriptAngular
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75