cloudfour / cloudfour/SimpleSlideView

Another little code question (variable: top)

Aperta
#18 1 commento 0 reazioni 1 assegnatario Rivendicata da @tylersticka Vedi su GitHub
question
Lingua principale
HTML
Stelle
257
Fork
31
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Hiya!

This line in `lib/simpleslideview.js`: https://github.com/cloudfour/SimpleSlideView/blob/master/lib/simpleslideview.js#L266

Is in the `beforeChangeEnd` callback:

``` javascript
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!

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.