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

你好 l-preview这个组件是你写的吗 #26

Open
openfe-openfe opened this issue Jan 13, 2025 · 12 comments
Open

你好 l-preview这个组件是你写的吗 #26

openfe-openfe opened this issue Jan 13, 2025 · 12 comments

Comments

@openfe-openfe
Copy link

这个npm包可以开源吗 想看看里面具体livephoto的实现

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025 via email

@openfe-openfe
Copy link
Author

好的 谢谢 找到啦

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025

live-photo
l-preview

@openfe-openfe
Copy link
Author

我们最近也有一个播放livephoto的需求,我看您详情页面这个是自己实现的,没有用livephotoskitjs。
看起来体验很好,为什么详情页面不用livephotoskitjs呢,是有什么坑吗。移动端您有测试吗

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025

我们最近也有一个播放livephoto的需求,我看您详情页面这个是自己实现的,没有用livephotoskitjs。

看起来体验很好,为什么详情页面不用livephotoskitjs呢,是有什么坑吗。移动端您有测试吗

本身livephotoskitjs已经可以满足基础的需求了(包括移动端touch相关的操作预览),只是对于想自定义额外的控制项不太方便,所有我这边是尝试写一个自定义程度大一点,可玩性高一点的产物,所以探索自己写一个来替代

@openfe-openfe
Copy link
Author

嗯嗯 我就想如果移动端 有些浏览器如果检测是video标签,是不是会渲染成一个视频播放器。我这就去试试,谢谢

@openfe-openfe
Copy link
Author

我这边也想自定义左上角icon等。livephotoskitjs这种覆盖样式可能有点麻烦

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025

如果自己写确实要考虑到移动端设备类似的video行为,特别是iPhone,小细节比较多,加油

@openfe-openfe
Copy link
Author

果然移动端 百度浏览器把video强制渲染成自己的播放器了
image

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025

尝试阻止一下默认行为

@openfe-openfe
Copy link
Author

不太行 老老实实用 livephotoskitjs了 这个我看是用canvas画的视频

@IceyWu
Copy link
Member

IceyWu commented Jan 13, 2025

我写的那一版应该是阻止了默认行为了的,然后canvas版本我也尝试了一下,在canvas分支,但说实话,蛮多细节,如果要求不高,还是推荐就用苹果那个

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

No branches or pull requests

2 participants