angular-ui/ui-grid

Question about grid.getVisibleRows()

Open

#6 294 ouverte le 4 juil. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
good first issue

Métriques du dépôt

Stars
 (5 395 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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? :)

Guide contributeur

Direction de recherche
Le problème remet en question le comportement de grid.getVisibleRows() dans angular ui grid. La ligne 5560 renvoie this.renderContainers.body.visibleRowCache, mais l'utilisateur suggère qu'elle devrait utiliser row.visible. Il n'y a pas de réponse des mainteneurs ni de PR existants. Étudiez l'implémentation actuelle dans le code source du dépôt (par exemple, Grid.js) pour déterminer si visibleRowCache représente correctement les lignes visibles, en tenant compte de la pagination, et vérifiez si le problème persiste dans les versions récentes. L'objectif est de confirmer s'il s'agit d'un bug et de proposer un correctif si nécessaire.
Stack technique
javascript
Domaine
frontend
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80