xfangfang / wiliwili

第三方B站客户端,目前可以运行在PC全平台、PSVita、PS4 、Xbox 和 Nintendo Switch上
https://xfangfang.github.io/wiliwili
GNU General Public License v3.0
3.11k stars 145 forks source link

WIP: add more share options #205

Closed happytommyl closed 6 months ago

happytommyl commented 11 months ago

I've add some share icons but I think the layout still need some adjustments. And I have some troubles as described in the related issues. #201

I'm still working on other platform specific implementations.

❤️

xfangfang commented 11 months ago

我就在这里写中文啦,因为这个项目应该是大部分都是中文参与。

  1. 触控的支持,可以参考:https://github.com/xfangfang/wiliwili/blob/7541334784c96991d6d391a5acda5410a875f1af/wiliwili/source/view/video_view.cpp#L244-L249

使用 addGestureRecognizer 的方式添加触控识别,borealis的clickAction只是做了按键的响应,鼠标和触摸是单独的,通过 addGestureRecognizer 来响应的。

  1. 点击非复制链接的按钮,可以直接跳转浏览器即可,应该和复制文本没有关系。

  2. 我建议不要在wiliwili里完成复制文本的代码,而是把复制文本的功能放到borealis的platform里去做,那里放置了很多平台相关的功能。

比如:打开浏览器

https://github.com/xfangfang/borealis/blob/6cf1fe70000d1db9ccceb50d93ef5eab20c7d9c6/library/lib/platforms/desktop/desktop_platform.cpp#L847-L863

模仿这个功能去实现即可,可以先给borealis完成相关的功能,提交一个PR,等那个合并之后再来做wiliwili这边的更新。

  1. 同一个PR尽量不要提交其他不相关的东西,比如 mongoose 和 cpr 的内容。(我猜这个还是因为分支切换错误提交的)
xfangfang commented 6 months ago

先将这个 pr关闭了,https://github.com/xfangfang/wiliwili/pull/277 解决了这个问题。