cloudfour / cloudfour/SimpleSlideView

Another little code question (variable: top)

Abierto
#18 1 comentario 0 reacciones 1 asignado Reclamado por @tylersticka Ver en GitHub
question
Lenguaje dominante
HTML
Estrellas
257
Forks
31
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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!

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.