cloudfour / cloudfour/SimpleSlideView

Another little code question (variable: top)

Ouverte
#18 1 commentaire 0 réactions 1 personne assignée Réclamée par @tylersticka Voir sur GitHub
question
Langage dominant
HTML
Étoiles
257
Forks
31
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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!

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.