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

[Feature]: 相册是否支持显示文件名呢? #1050

Closed
3 tasks done
mouyong opened this issue Dec 28, 2022 · 21 comments
Closed
3 tasks done

[Feature]: 相册是否支持显示文件名呢? #1050

mouyong opened this issue Dec 28, 2022 · 21 comments
Assignees

Comments

@mouyong
Copy link

mouyong commented Dec 28, 2022

前置阅读 | Pre-reading

PicGo的版本 | PicGo Version

v2.3.1

系统信息 | System Information

Mac(arm64)

功能请求 | Feature request

期望相册支持显示文件名,以及上传的不是可预览的图片时,能有默认的 placeholder 图片。

image

@Molunerfinn
Copy link
Owner

会考虑实现,感谢反馈

@mouyong
Copy link
Author

mouyong commented Dec 28, 2022

pr 提交了,可以参考一下。下面是预览图

image

image

@mouyong
Copy link
Author

mouyong commented Dec 28, 2022

image

不行。还得大佬修复。我无法完成复制。第一个文件上传后,点击不到复制按钮。

@mouyong
Copy link
Author

mouyong commented Dec 28, 2022

image

点击复制的按钮之后很久才提示复制成功。不知道为啥。

@Molunerfinn
Copy link
Owner

回头我处理一下

@Molunerfinn
Copy link
Owner

Molunerfinn commented Dec 29, 2022

我才你这里上传的并非图片格式的文件,这样会导致图片标签加载不符合预期,同时notification里因为要加载图片会加载成如你图示里的视频文件,会导致下载时间过长。PicGo设计之初,相册只用于加载图片。所以别的文件不在相册里显示是符合预期的。

这里需要统一处理这种情况,非图片文件的相册加载。我后续自行处理

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

没关系的,需要的功能其实只是显示出上传过这个文件。以及文件的名字。

对了,上传完成能支持通知 webhook 地址不?

还有个问题。1G以上的大文件上传和下载有比较好的方案吗。

因为是内网线上线下组网的,大文件上传会有点慢,下载也有点慢。

目前是做了一套转码程序,大视频上传到内网后,转码程序转成m3u8进行点播,然后在线web后台设置视频的资讯信息,就可以在小程序加载了。

@Molunerfinn
Copy link
Owner

Molunerfinn commented Dec 30, 2022

picgo初衷不是用来做大文件上传下载的,这方面有更专业的工具,你可以尝试搜索。以及你的问题我并不了解。同时PicGo也不支持断点续传。所以这块我不能给到答案,因为我也没做过。

上传完成后的功能,你可以自行开发插件实现。https://picgo.github.io/PicGo-Core-Doc/zh/dev-guide/cli.html#afteruploadplugins

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

有地方可以查看 picgo 的 node 版本吗,我想确定下是不是因为版本高了导致无法进行 dev 预览开发。

我的 node 默认版本是 v16.14.0, nvm 版本是 0.39.1, yarn 版本是 1.22.19

@Molunerfinn
Copy link
Owner

node 16就行。dev开发时,需要把安装版退出

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

https://github.com/zjsxdyw/file-chunk-upload 大文件上传下载的方案有了。参考借鉴这个。

服务器上运行项目,然后nginx反向代理,隐藏端口,然后客户端再完成上传,然后 webhook 通知。就达到了接入系统使用的目的。

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

有退出呢,启动yarn dev后,mac 的程序坞有个 electron 软件。但是没有开启任何程序。菜单没东西。

今天运行,直接提示报错。不知道咋解决,前几天还可以写代码后 build。但是效率很低。

image

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

移除项目目录下的 main 后,执行效果如下图

image

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

噢,没退出导致的。

退出后,运行了,看不到界面。只有个图标在下方

image

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

有了。估计是之前菜单栏图标太多了,然后被摄像头挡住了,所以看不到

image

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

显示文件名这个太重要了,用在区分文件方面很方便。其他类型的文件,期望可以有个默认的占位图。

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

期望能在上传的时候切换同一个插件下的其他配置。这样方便将不同的文件上传到不同的地方。然后 menu icon 也能支持切换操作,这样不用打开主窗口,使用会更方便。

image

image

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

menu 的上传等待区。粘贴板的图片预览操作。如果图片过大,会加载好一会。建议放按钮,粘贴板图片,点击后触发上传就可以了。

@mouyong
Copy link
Author

mouyong commented Dec 30, 2022

https://file.keking.cn/

通过集成这个kkfileview,可以达到预览绝大多数文件的目的。不知道相册的位置能通过插件增加按钮吗。

示例:
https://file.keking.cn/onlinePreview?url=aHR0cDovL2ZpbGVzLmhlY3MuaXdud2ViLmNvbS9vdXRzaXRlX2xpbmsvMjAyMjEyMzEwMDExODUzLnBuZw%3D%3D

@Molunerfinn
Copy link
Owner

https://file.keking.cn/

通过集成这个kkfileview,可以达到预览绝大多数文件的目的。不知道相册的位置能通过插件增加按钮吗。

示例:

https://file.keking.cn/onlinePreview?url=aHR0cDovL2ZpbGVzLmhlY3MuaXdud2ViLmNvbS9vdXRzaXRlX2xpbmsvMjAyMjEyMzEwMDExODUzLnBuZw%3D%3D

不考虑。后续会开放插件接入webview,插件自行可以绘制界面。本体相册不做复杂逻辑。

@Molunerfinn
Copy link
Owner

Molunerfinn commented Dec 31, 2022

menu 的上传等待区。粘贴板的图片预览操作。如果图片过大,会加载好一会。建议放按钮,粘贴板图片,点击后触发上传就可以了。

放图片是为了二次确认,图片过大会导致主进程和渲染进程传输数据大,这个无法避免。如果你已经很确定要上传的图片,可以直接使用快捷键上传,可以节省很多时间。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants