cloudfour / cloudfour/SimpleSlideView

Another little code question (variable: top)

オープン
#18 コメント 1 件 リアクション 0 件 担当者 1 名 @tylersticka が担当を希望しています GitHub で見る
question
主要言語
HTML
スター
257
フォーク
31
PR マージ指標
30日以内にマージされた PR はありません

説明

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!

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。