angular-ui/ui-grid

Select All check box not getting checked when disabled rows present

Open

#6.624 geöffnet am 22. März 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
good first issuetype: enhancement

Repository-Metriken

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

Beschreibung

I am using the version 4.1.0 of ui-grid. I have enabled the row selection header.

I have two rows in the grid. One row is disabled using isRowSelectable(). Now if i check select all, the enabled row gets selected and the disabled row is not selected. That's fine.

Now if i unselect that enabled row, select all checkbox too gets unselected. That too fine.

But now if i select that enabled row alone, select all checkbox is not checked. It has only two rows and in that one enabled row is checked, So here i expect select all check box to be checked.

http://plnkr.co/edit/A6r6sfzA3CJhZp4SulhQ?p=preview

Contributor Guide

Research-Richtung
Prüfen Sie die Auswahllogik in ui grid, um zu verstehen, wie der Zustand der Alle auswählen Checkbox berechnet wird, wenn deaktivierte Zeilen vorhanden sind. Verfolgen Sie den Code, der den aktivierten Zustand basierend auf ausgewählten Zeilen im Vergleich zur Gesamtzahl der auswählbaren Zeilen bestimmt.
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
GitAngular
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75