angular-ui/ui-grid

emptyBaseLayer.directive and expandable.directive collision

Open

#6 830 ouverte le 8 août 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
good first issuegrid-empty-base-layer

Métriques du dépôt

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

Description

Because of the same priority with emptyBaseLayer.directive, expandable.directive on line: var rowRepeatDiv = angular.element($elm.children().children()[0]);

Gets the rowRepeatDiv of wrong element: div.ui-grid-empty-base-layer-container.ui-grid-canvas instead of: div.ui-grid-canvas (which is the second element of the array - $elm.children().children()[1])

Becouse of that, sub-rows wont get rendered. Raising expandable.directive priority fixes the rendering.

Is it a good case for a pullrequest? Or is it because of my configuration? I also use:

  • ui-grid-pagination
  • ui-grid-auto-resize
  • ui-grid-selection

I tried ui-grid-empty-base-layer="false", but that didn't help.

Guide contributeur

Direction de recherche
Étudiez les priorités des directives dans ui grid, en comparant spécifiquement emptyBaseLayer.directive et expandable.directive. Le problème suggère d'augmenter la priorité de expandable.directive pour corriger la collision. Confirmez en examinant les définitions des directives et la structure DOM dans la grille. Vérifiez si la correction introduit des effets secondaires pour d'autres fonctionnalités comme la pagination ou le redimensionnement automatique. Reportez vous au problème lié et aux commentaires pour un contexte supplémentaire.
Stack technique
javascriptangular
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
AngularGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75