angular-ui/ui-grid

emptyBaseLayer.directive and expandable.directive collision

Open

#6.830 geöffnet am 8. Aug. 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
good first issuegrid-empty-base-layer

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Direktivenprioritäten in ui grid, insbesondere vergleiche emptyBaseLayer.directive und expandable.directive. Das Problem schlägt vor, die Priorität von expandable.directive zu erhöhen, um die Kollision zu beheben. Bestätige dies durch Prüfung der Direktivendefinitionen und der DOM Struktur im Grid. Überprüfe, ob der Fix Nebenwirkungen für andere Funktionen wie Paginierung oder automatische Größenanpassung einführt. Siehe das verlinkte Issue und die Kommentare für weiteren Kontext.
Tech Stack
javascriptangular
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
AngularGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75