Cp0204 / quark-auto-save

夸克网盘签到、自动转存、命名整理、发推送提醒和刷新媒体库一条龙
GNU Affero General Public License v3.0
445 stars 69 forks source link

添加从指定文件开始订阅的功能,优化前端展示 #35

Closed lustlost closed 1 month ago

lustlost commented 1 month ago

添加从指定文件开始订阅的功能,优化前端展示

lustlost commented 1 month ago

前端优化:订阅项目添加折叠效果 功能增加:同时增加【文件开始】功能,可以选择指定文件,订阅将从指定的文件之后更新的文件开始,应对一些剧很多集,一般从最新没看过的集数开始订阅就行了 效果如下 image

Cp0204 commented 1 month ago

首先感谢PR!

“添加从指定文件开始订阅的功能” 没看懂逻辑,根据代码这也只是跳过某个fid的文件而已啊。

另外,这个PR除了提到功能以外,引入了许多非必要文件,具有侵入性;并且多处非必要地改动了原有代码风格,这是不利于原作者维护的。故暂不合并,Salute!

lustlost commented 1 month ago

首先感谢PR!

“添加从指定文件开始订阅的功能” 没看懂逻辑,根据代码这也只是跳过某个fid的文件而已啊。

另外,这个PR除了提到功能以外,引入了许多非必要文件,具有侵入性;并且多处非必要地改动了原有代码风格,这是不利于原作者维护的。故暂不合并,Salute!

是跳过了从某个fid之后的所有文件,因为文件列表是按时间排序的,所以逻辑就是按时间从新到老的排序后,从某个文件之前的文件都排除,如果一步动漫有几百集,大部分情况只要从没看过的开始订阅就可以,可以大大节省网盘空间(具体可以参考aliyundrive-subscribe包含此功能) 非不要文件是一开始没有ignore python虚拟环境,这个PR可以不合并,但希望能增加【文件开始】功能 另外不合并此PR的话,同时考虑一下前端优化一下。也可以wx 或qq联系交流

Cp0204 commented 1 month ago

是跳过了从某个fid之后的所有文件

明白了,感觉这个处理逻辑也不太对。文件列表目前是按时间倒序排序,但是集数顺序不一定按时间排序,例如03.mp4是有可能比01.mp4上传时间早,批量上传并且文件大小有差异时,这种情况很常见。而且批量转存时,文件时间是按转存时间,也都是乱的。

要实现的话,改为按文件名排序会比较好。

Cp0204 commented 1 month ago

这两个功能点都挺好的,只是改得比较乱 😂 我怕合并完我都对项目代码陌生了

也给你一点点建议,一个commit一个优化点,给其他项目pr的话不做代码格式的大改,只改片段

你要是不介意的话,我直接copy你的代码片段来改

lustlost commented 1 month ago

这两个功能点都挺好的,只是改得比较乱 😂 我怕合并完我都对项目代码陌生了

也给你一点点建议,一个commit一个优化点,给其他项目pr的话不做代码格式的大改,只改片段

你要是不介意的话,我直接copy你的代码片段来改

大部分动漫追剧都是一周一集更新,按照时间排序基本上不会错,进一步优化可以用户自己定义排序方式是按时间还是按文件名称。你直接copy不合并这个PR我不介意,主要想官方版本能直接集成这个功能就行

还有前端还是希望能优化一下,类似cookie、通知、emby这些不常用的功能可以移到最下面,任务可以用collapse折叠一下,就很完美了

Cp0204 commented 1 month ago

前端昨晚已经改了,还要兼顾一些响应式UI,你可以拉 main 标签看看。 还是把cookie放在上面,我考虑的是操作动线和主次的问题,新用户看到页面应该先输入什么,再输入什么,应该有直观的感受,这也是要考虑的。

从某文件开始订阅,我打算结合文件名排序来改。

还有一个一直想做但懒得做的,在前端中点选保存路径。既然列文件都做了,也可以顺着去做。

Cp0204 commented 1 month ago

搞定,暂时按你的逻辑,文件修改时间倒序;我把break放到for的最后,这样才包含选中的文件。