HMCL icon indicating copy to clipboard operation
HMCL copied to clipboard

添加对整合包可选文件的初步支持

Open CaveNightingale opened this issue 3 years ago • 57 comments

RT,modrinth和curseforge整合包格式均支持声明可选模组/文件,而HMCL会直接将这些文件当作必选处理 mcbbs的整合包格式我目前找不到文档,故暂不做改动 此次改动会导致CurseForge导入界面因下载文件信息而加载时间变长,对Modrinth无影响

CaveNightingale avatar Oct 03 '22 11:10 CaveNightingale

Merge Conflict 了,记得改一改

burningtnt avatar May 23 '23 12:05 burningtnt

ping @CaveNightingale 麻烦把这个 PR 更新一下

zkitefly avatar Jul 29 '23 14:07 zkitefly

人已经 9 个月不见了,大概是跑了(

burningtnt avatar Jul 29 '23 14:07 burningtnt

ping @CaveNightingale 麻烦把这个 PR 更新一下

好的

CaveNightingale avatar Jul 29 '23 17:07 CaveNightingale

《跑了》

hejiehao avatar Jul 30 '23 02:07 hejiehao

完成

CaveNightingale avatar Jul 30 '23 08:07 CaveNightingale

@huanghongxun 这个已经一年没合了

zkitefly avatar Aug 07 '23 10:08 zkitefly

功能请求

要不加个全选(或全不选)按钮?

要不加个全选(或全不选)按钮?

zkitefly avatar Dec 31 '23 00:12 zkitefly

Curse 整合包导入测试

正在通过网络获取文件名

加载成功画面

当时未加 CURSEFORGE_API_KEY 时的图片

注:未测试安装

RLCraft 1.12.2 - Release v2.9.3(改造版).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

问题:若网络环境不佳,Curse 整合包下面的文件名显示是直接为空白,我觉得这个文件名获取可能会有点问题?

zkitefly avatar Dec 31 '23 00:12 zkitefly

Modrinth 整合包导入测试

加载成功画面

注:未测试安装

Cobblemon Modpack [Fabric] 1.4.1(改造版).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

建议:我发现下方的文件选择没标题可能会让用户不知道是什么东西,我建议在上面加个标题

zkitefly avatar Dec 31 '23 01:12 zkitefly

Modrinth 整合包安装测试

files 列表:

fancymenu_fabric_2.14.10-2_MC_1.20.1.jar # 可选
lazydfu-0.1.3.jar # 必选
notenoughanimations-fabric-1.6.4-mc1.20.jar # 必选
krypton-0.2.3.jar # 必选
Xaeros_Minimap_23.9.3_Fabric_1.20.jar # 可选
cloth-config-11.1.118-fabric.jar # 可选

导入页面


全选 可选 项目的安装页面

选择全选 可选 项目安装成功后的模组列表页面


全不选 可选 项目的安装页面

选择全选 可选 项目安装成功后的模组列表页面


Cobblemon Modpack [Fabric] 1.4.1(改造+精简).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

zkitefly avatar Dec 31 '23 01:12 zkitefly

请将加载整合包文件的 Task 显示至屏幕上,并以并发操作

burningtnt avatar Dec 31 '23 01:12 burningtnt

请将加载整合包文件的 Task 显示至屏幕上,并以并发操作

我觉得不太行,可能会影响操作流畅性

zkitefly avatar Dec 31 '23 02:12 zkitefly

请将加载整合包文件的 Task 显示至屏幕上,并以并发操作

我觉得不太行,可能会影响操作流畅性

那就添加一个 Spinner,让用户感知到HMCL 正在加载

burningtnt avatar Dec 31 '23 04:12 burningtnt

此外,建议把下面的仅文件名改为模组下载界面的 UI 风格,即,可以点进去查看详情

burningtnt avatar Dec 31 '23 04:12 burningtnt

@CaveNightingale ping

zkitefly avatar Dec 31 '23 08:12 zkitefly

@CaveNightingale ping

CaveNightingale avatar Dec 31 '23 09:12 CaveNightingale

请将加载整合包文件的 Task 显示至屏幕上,并以并发操作

image

他看错了,所以不需要这样了

zkitefly avatar Dec 31 '23 09:12 zkitefly

主要还是这一条:

此外,建议把下面的仅文件名改为模组下载界面的 UI 风格,即,可以点进去查看详情

和:

问题:若网络环境不佳,Curse 整合包下面的文件名显示是直接为空白,我觉得这个文件名获取可能会有点问题?

建议改成:每一个可选模组为一个 TwoLineListItem,可参考模组下载界面,如果失败,则显示“失败,点击重试”

burningtnt avatar Dec 31 '23 09:12 burningtnt

我认为并不认同 改为模组下载界面的 UI 风格,就原来这个挺好的

如果有模组名称获取失败就重试几遍(5遍就够了),还是不行就直接贴一个 [加载失败,点击重试] 的一个小按钮

然后这个可选模组页面,加一个全选(全不选)按钮,然后加个 可选模组 标题我觉得就够了

zkitefly avatar Dec 31 '23 10:12 zkitefly

我认为并不认同 改为模组下载界面的 UI 风格,就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息,那就需要改为我所提的这种风格了 image 类似这种 ↑

burningtnt avatar Dec 31 '23 10:12 burningtnt

这样会不会增加复杂度啊

zkitefly avatar Dec 31 '23 10:12 zkitefly

#1771 #1771-2

zkitefly avatar Dec 31 '23 10:12 zkitefly

此外,建议把下面的仅文件名改为模组下载界面的 UI 风格,即,可以点进去查看详情

我认为并不认同 改为模组下载界面的 UI 风格,就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息,那就需要改为我所提的这种风格了 image 类似这种 ↑

虽然但是,好像整合包文件里本来就没有模组的详细信息啊?

CaveNightingale avatar Dec 31 '23 10:12 CaveNightingale

这样会不会增加复杂度啊

如果移动到单独界面呢?即:

  • 将原来显示“加载可选模组”哪个地方删除
  • 在安装左侧添加一个加载条,并在加载完毕后变为一个按钮,点击后进入新的界面展示模组列表并供用户勾选

用户只看一个模组的文件名,是无法考虑要还是不要这个模组的,确实需要展示模组详细信息

加油

burningtnt avatar Dec 31 '23 10:12 burningtnt

By the way,请问你这里的 RT 是指?

RT,modrinth和curseforge整合包格式均支持声明可选模组/文件,而HMCL会直接将这些文件当作必选处理

burningtnt avatar Dec 31 '23 10:12 burningtnt

此外,建议把下面的仅文件名改为模组下载界面的 UI 风格,即,可以点进去查看详情

我认为并不认同 改为模组下载界面的 UI 风格,就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息,那就需要改为我所提的这种风格了 image 类似这种 ↑

虽然但是,好像整合包文件里本来就没有模组的详细信息啊?

Curse 整合包可以获取

hejiehao avatar Dec 31 '23 10:12 hejiehao

虽然但是,好像整合包文件里本来就没有模组的详细信息啊?

你都拿到 Project ID 了,不就啥都有了

https://docs.modrinth.com/modpacks/format 但是隔壁Modrinth只给下载连接

CaveNightingale avatar Dec 31 '23 10:12 CaveNightingale

By the way,请问你这里的 RT 是指?

RT,modrinth和curseforge整合包格式均支持声明可选模组/文件,而HMCL会直接将这些文件当作必选处理

如题

CaveNightingale avatar Dec 31 '23 10:12 CaveNightingale

虽然但是,好像整合包文件里本来就没有模组的详细信息啊?

你都拿到 Project ID 了,不就啥都有了https://docs.modrinth.com/api-spec#tag/version-files/operation/versionFromHash

https://docs.modrinth.com/modpacks/format 但是隔壁Modrinth只给下载连接

可以通过 hash 获取吗? https://docs.modrinth.com/api-spec#tag/version-files/operation/versionFromHash https://docs.modrinth.com/api-spec#tag/version-files/operation/versionsFromHashes

hejiehao avatar Dec 31 '23 10:12 hejiehao