Another little code question (variable: top)
@tylersticka arbeitet bereits daran.
Seit 24.4.2014.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
Hiya!
This line in lib/simpleslideview.js: https://github.com/cloudfour/SimpleSlideView/blob/master/lib/simpleslideview.js#L266
Is in the beforeChangeEnd callback:
beforeChangeEnd = function() {
if (_this.options.resizeHeight) {
if (_this.options.maintainViewportHeight && window.innerHeight > _this.lastViewportHeight) {
_this.lastViewportHeight = window.innerHeight;
$html.css('min-height', (_this.lastViewportHeight + top) + 'px');
}
return _this.$container.animate({
height: outerHeight($targetView)
}, _this.options.heightDuration, _this.options.easing, onChangeEnd);
} else {
return onChangeEnd();
}
};
top isn't defined. Not sure what it should be so need a bit of input to fix it!
- Vorherrschende Sprache
- HTML
- Sterne
- 257
- Forks
- 31
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus cloudfour/SimpleSlideView
-
Normalizing some options now that noZeptoEvtl. wieder frei @tylersticka hat das vor 4579 Tagen übernommen, und es ist kein Pull Request offen. Offenquestion
cloudfour/SimpleSlideView#16 · 1 Kommentar · 1 zugewiesene Person ·
-
enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
cloudfour/SimpleSlideView#10 · 4 Kommentare ·
-
enhancement
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
-
history support (a.k.a. back button support)Evtl. wieder frei @tylersticka hat das vor 4843 Tagen übernommen, und es ist kein Pull Request offen. Offenenhancement
cloudfour/SimpleSlideView#5 · 5 Kommentare · 1 zugewiesene Person ·