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

refactor(NCarousel): refactor carousel #2024

Closed
wants to merge 5 commits into from
Closed

refactor(NCarousel): refactor carousel #2024

wants to merge 5 commits into from

Conversation

nooooooom
Copy link
Contributor

还有些地方没有处理完，先提交上来看看效果

@vercel
Copy link

vercel bot commented Dec 24, 2021

@nooooooom is attempting to deploy a commit to the Tusimple Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Dec 24, 2021 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tusimple/naive-ui/2mhETcCfEJAS6YAQRpxLgq767ZYR
✅ Preview: https://naive-ui-git-fork-nooooooom-feat-carousel-resize-tusimple.vercel.app

@07akioni
Copy link
Collaborator

熬夜够狠的

@nooooooom
Copy link
Contributor Author

nooooooom commented Dec 24, 2021 •
edited
Loading

熬夜够狠的

😭明天还得加班，今天得先睡了，还有一些要调的

@07akioni
Copy link
Collaborator

本来想弄个小船被你改成航空母舰了

@nooooooom
Copy link
Contributor Author

本来想弄个小船被你改成航空母舰了

希望我没用弄错方向，尽量在可控制的范围内增加一些常用的功能

@XieZongChen
Copy link
Collaborator

image
示例貌似重复了

@nooooooom
Copy link
Contributor Author

image 示例貌似重复了

是的哥，还没有调完，放些示例上来让大家提提意见

@XieZongChen
Copy link
Collaborator

#1882 #1892 #1931 验证一下是否能在这次重构中得到解决

@nooooooom
Copy link
Contributor Author

nooooooom commented Dec 25, 2021 •
edited
Loading

#1882 #1892 #1931 验证一下是否能在这次重构中得到解决

所有关于 Carousel 提的bug或需求都能解决，我今天或明天完善之后重新开个PR提上来

@Sepush
Copy link
Collaborator

Sepush commented Dec 25, 2021

非常棒的工作🚀🚀🚀
有几个小问题

  1. 居中的那个demo 想 aoto paly一样 当滚动到最后一张的时候在滚动自动切换到第一张
  2. 居中的时候当滚的到最左边的图片 如果用鼠标拖动 会有残影 具体效果如下图
    image
  3. 鼠标滚轮控制好像不work 当点击轮播在滚动滚轮 是页面滚动

@nooooooom
Copy link
Contributor Author

非常棒的工作🚀🚀🚀 有几个小问题

  1. 居中的那个demo 想 aoto paly一样 当滚动到最后一张的时候在滚动自动切换到第一张
  2. 居中的时候当滚的到最左边的图片 如果用鼠标拖动 会有残影 具体效果如下图
    image
  3. 鼠标滚轮控制好像不work 当点击轮播在滚动滚轮 是页面滚动

我超，我没复现出来。
居中那个DOM是不是拖拽事件失焦影响了？我对拖拽物做了近中线的判断，估计是又拖回第一个去了。
鼠标滚轮没做什么调控，根据浏览器的规则貌似鼠标停留个几百毫秒就会将滚轮的焦点放置在鼠标位置的DOM上，还需要调控的是只有动画结束（切换真正完成）后才进行下一次的鼠标滚动响应。

@Sepush
Copy link
Collaborator

Sepush commented Dec 25, 2021

非常棒的工作🚀🚀🚀 有几个小问题

  1. 居中的那个demo 想 aoto paly一样 当滚动到最后一张的时候在滚动自动切换到第一张
  2. 居中的时候当滚的到最左边的图片 如果用鼠标拖动 会有残影 具体效果如下图
    image
  3. 鼠标滚轮控制好像不work 当点击轮播在滚动滚轮 是页面滚动

我超，我没复现出来。 居中那个DOM是不是拖拽事件失焦影响了？我对拖拽物做了近中线的判断，估计是又拖回第一个去了。 鼠标滚轮没做什么调控，根据浏览器的规则貌似鼠标停留个几百毫秒就会将滚轮的焦点放置在鼠标位置的DOM上，还需要调控的是只有动画结束（切换真正完成）后才进行下一次的鼠标滚动响应。

拖拽那个preview里稳定复现 只有左边会触发

Carousel.-.Naive.UI.-.Google.Chrome.2021-12-26.00-11-15.mp4

@nooooooom
Copy link
Contributor Author

非常棒的工作🚀🚀🚀 有几个小问题

  1. 居中的那个demo 想 aoto paly一样 当滚动到最后一张的时候在滚动自动切换到第一张
  2. 居中的时候当滚的到最左边的图片 如果用鼠标拖动 会有残影 具体效果如下图
    image
  3. 鼠标滚轮控制好像不work 当点击轮播在滚动滚轮 是页面滚动

我超，我没复现出来。 居中那个DOM是不是拖拽事件失焦影响了？我对拖拽物做了近中线的判断，估计是又拖回第一个去了。 鼠标滚轮没做什么调控，根据浏览器的规则貌似鼠标停留个几百毫秒就会将滚轮的焦点放置在鼠标位置的DOM上，还需要调控的是只有动画结束（切换真正完成）后才进行下一次的鼠标滚动响应。

拖拽那个preview里稳定复现 只有左边会触发

Carousel.-.Naive.UI.-.Google.Chrome.2021-12-26.00-11-15.mp4

可能我卡浏览器的渲染了，可能样式计算里面的副作用太多了，明天搞完它

@Sepush
Copy link
Collaborator

Sepush commented Dec 25, 2021 •
edited
Loading

非常棒的工作🚀🚀🚀 有几个小问题

  1. 居中的那个demo 想 aoto paly一样 当滚动到最后一张的时候在滚动自动切换到第一张
  2. 居中的时候当滚的到最左边的图片 如果用鼠标拖动 会有残影 具体效果如下图
    image
  3. 鼠标滚轮控制好像不work 当点击轮播在滚动滚轮 是页面滚动

我超，我没复现出来。 居中那个DOM是不是拖拽事件失焦影响了？我对拖拽物做了近中线的判断，估计是又拖回第一个去了。 鼠标滚轮没做什么调控，根据浏览器的规则貌似鼠标停留个几百毫秒就会将滚轮的焦点放置在鼠标位置的DOM上，还需要调控的是只有动画结束（切换真正完成）后才进行下一次的鼠标滚动响应。

拖拽那个preview里稳定复现 只有左边会触发
Carousel.-.Naive.UI.-.Google.Chrome.2021-12-26.00-11-15.mp4

可能我卡浏览器的渲染了，可能样式计算里面的副作用太多了，明天搞完它

小问题 慢慢修 弄完咱们的轮播就牛了

@nooooooom
feat(Carousel):
98730df 
feat: support custom dots
feat: support custom arrow
fix:  fix vnode mess on loop status switch
fix: wrong size calculation on `spaceBetween` props
perf: reduce the impact of reactivity effects
chore: don't slide to when wheel
@nooooooom
Copy link
Contributor Author

感觉功能差不多了，明天完善一下demo跟测试用例就可以真正提上去了

@07akioni
Copy link
Collaborator

不容易

@nooooooom nooooooom closed this Dec 27, 2021
@nooooooom nooooooom deleted the feat-carousel-resize branch February 16, 2022 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nooooooom @07akioni @XieZongChen @Sepush