Another little code question (variable: top)
オープン
@tylersticka がすでに取り組んでいます。
2014年4月24日 から。
評価
この issue はまだ評価されていません。
説明
question
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!
- 主要言語
- HTML
- スター
- 257
- フォーク
- 31
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cloudfour/SimpleSlideView のほかの issue
-
Normalizing some options now that noZepto再び着手できるかも @tylersticka が 4578 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンquestion
cloudfour/SimpleSlideView#16 · コメント 1 件 · 担当者 1 名 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
cloudfour/SimpleSlideView#10 · コメント 4 件 ·
-
enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
-
history support (a.k.a. back button support)再び着手できるかも @tylersticka が 4842 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンenhancement
cloudfour/SimpleSlideView#5 · コメント 5 件 · 担当者 1 名 ·