Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

参数circular设置后滚动效果错误 #29

Closed
nuintun opened this issue Oct 18, 2013 · 7 comments
Closed

参数circular设置后滚动效果错误 #29

nuintun opened this issue Oct 18, 2013 · 7 comments

Comments

@nuintun
Copy link

nuintun commented Oct 18, 2013

#27 还是这个问题，今天终于明白哪里不对了,看图先~

circular
按右边的>进行操作，画面向左一直循环，这里没有问题！
然后测试左边<按钮，理论和用户感觉上都应该是画面向右一直循环，但是现在确是：当切换首页和尾页的时候滚动方向会改变，而且会一次性滚动几个屏幕！
纠结啊~
@lizzie lizzie closed this as completed in 13c58cd Oct 21, 2013
@lizzie
Copy link
Member

lizzie commented Oct 21, 2013

改了, 例子见 http://aralejs.org/switchable/examples/carousel.html#使用-carousel-实现的-slide

@nuintun
Copy link
Author

nuintun commented Oct 22, 2013

😄好，Thanks~

@nuintun
Copy link
Author

nuintun commented Oct 22, 2013

@lizzie circular对貌似对分页滚动无效啊！看图~
qq20131022105409
按照下一页的逻辑点击分页1>2>3>4>5>1>2>3>4>5循环是正确的。
按照上一页的逻辑点击分页5>4>3>2>1>5>4>3>2>1循环就出现了和fix之前一样的错误~
点击分页切换的时候是否可以根据分页的index值来判断移动方向，当前页index-目标页index，如果大于0相当于下一页的移动方向，否则相当于上一页的效果，是0则不移动。

@lizzie
Copy link
Member

lizzie commented Oct 22, 2013

@newtonniu 上一页的逻辑没问题啊. 都是往前切换. 你是不是看了没 fix 的版本?

分页切换, 我还是觉得按照现在的逻辑比较好.

你再看看?

@nuintun
Copy link
Author

nuintun commented Oct 22, 2013

@lizzie 上一页没问题，用分页切换的时候有点问题，可以参考bxslider的方式~
你直接用分页切换1>5和5>1赶脚蛮怪异~，不过也不是大问题 😄

@lizzie
Copy link
Member

lizzie commented Oct 22, 2013

@newtonniu 好吧, 再一次提交了. 你看下最终的版本.

@nuintun
Copy link
Author

nuintun commented Oct 23, 2013

@lizzie 已经好了~ Perfect:smile:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lizzie @nuintun