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 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
cloudfour/SimpleSlideView 的其他 Issue
-
Normalizing some options now that noZepto可能重新可做 @tylersticka 于 4577 天前认领，目前没有进行中的 PR。 未关闭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 于 4841 天前认领，目前没有进行中的 PR。 未关闭enhancement
cloudfour/SimpleSlideView#5 · 5 条评论 · 已指派 1 人 ·