angular-ui/ui-grid

Question about grid.getVisibleRows()

Open

#6294 aperta il 4 lug 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2496 fork)batch import
good first issue

Metriche repository

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

Descrizione

According to the angular-ui-grid@4.0.6

Here is the code: (about line 5560)

Grid.prototype.getVisibleRows = function getVisibleRows() {
    return this.renderContainers.body.visibleRowCache;
};

Why visible rows from the visibleRowCache? Because the visibleRowCache just have the current page data. Therefore, we need to get visible rows by using row.visible, rather than grid.getVisibleRows. But i don't know whether it is a bug?

Will you pretty good guys to fix this problem? :)

Guida contributor

Direzione di ricerca
Il problema mette in discussione il comportamento di grid.getVisibleRows() in angular ui grid. La riga 5560 restituisce this.renderContainers.body.visibleRowCache, ma l'utente suggerisce che dovrebbe utilizzare row.visible. Non esistono risposte dai manutentori o PR. Indagare l'implementazione attuale nel codice sorgente del repository (ad es., Grid.js) per determinare se visibleRowCache rappresenta correttamente le righe visibili, considerando l'impaginazione, e verificare se il problema persiste nelle versioni recenti. L'obiettivo è confermare se si tratta di un bug e proporre una correzione se necessario.
Tech stack
javascript
Dominio
frontend
Tipo issue
Ricerca
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80