angular-ui/ui-grid

scrollToIfNecessary sets bottomBound as NaN

Geschlossen

#3.849 geöffnet am 24.06.2015

Auf GitHub ansehen
 (9 Kommentare) (1 Reaktion) (1 zugewiesene Person)JavaScript (2.496 Forks)batch import
help wantedneeds: analysis

Repository-Metriken

Stars
 (5.395 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In ui-grid - v3.0.0-rc.22 I am trying to execute the scrollTo functionality. When I hit line 5280:

var bottomBound = self.renderContainers.body.prevScrollTop + self.gridHeight - self.renderContainers.body.headerHeight - self.footerHeight - self.scrollbarWidth;

the value for 'self.renderContainers.body.headerHeight' is undefined which causes the 'bottomBound' to be set to 'NaN'. It seems like the value used here should instead be 'self.headerHeight' which is used to set the 'topBound' on line 5270.

Am I either incorrect in that the value should be switched or am I doing something else that causes self.renderContainers.body.headerHeight to be 'undefined'?

Contributor Guide

Research-Richtung
Überprüfen Sie die in scrollToIfNecessary verwendete Eigenschaft und erwägen Sie, self.renderContainers.body.headerHeight durch self.headerHeight zu ersetzen, um NaN zu vermeiden.
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
JavaScriptAngular
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70