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.

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

[组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法触发加载的问题 #4430

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

Tinhone
Copy link
Contributor

@Tinhone Tinhone commented Sep 12, 2023

fix #4302

如题,当 ScrollTrigger 初次触发加载且父级组件完成加载后,如果因为高分屏、低缩放比例等原因,使 ScrollTrigger 仍在视口内部的话,将会导致 IntersectionObserver 无法继续触发,接着导致组件无法自动触发加载,最后导致受影响用户遇到父级组件不会自动继续加载的问题。

无法继续触发 IntersectionObserver 的原因是,此时页面滚动不了,ScrollTrigger 无法离开和进入视口。

这个 PR 通过为 ScrollTrigger 增加可视判断逻辑来修复上述问题,同时可以解决 #4302 报告的情况。

由于问题描述有点区别,可能没有解决这条回复#4428 遇到的问题,需要后续反馈才能确认具体情况。

另外,为了避免可视判断逻辑造成的重复加载情况,这个修复方法还需要修改用到 ScrollTrigger 并且需要它的可视判断逻辑的组件代码,使其能够向 ScrollTrigger 传递加载状态。

考虑到一次性全改了可能会出现奇奇怪怪的问题,所以这个 PR 只调整了有问题报告的极简首页组件,可以先看看效果。

@Tinhone Tinhone changed the title [组件-极简首页] fix: 修复 ScrollTrigger 在初次加载后若仍在视口内就无法继续自动加载的问题 [组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法自动触发加载的问题 Sep 12, 2023
@Tinhone Tinhone changed the title [组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法自动触发加载的问题 [组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法触发加载的问题 Sep 12, 2023
@Tinhone Tinhone changed the title [组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法触发加载的问题 [组件-极简首页] fix: 修复 ScrollTrigger 在特定情况下无法触发加载的问题 Sep 13, 2023
@the1812 the1812 merged commit 97f4d61 into the1812:preview-fixes Nov 2, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants