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] 模组搜索页文件下载详情中的下载按钮 #2727

Open
zkitefly opened this issue Feb 2, 2024 · 11 comments · May be fixed by #2808
Open

[Feature] 模组搜索页文件下载详情中的下载按钮 #2727

zkitefly opened this issue Feb 2, 2024 · 11 comments · May be fixed by #2808

Comments

@zkitefly
Copy link
Member

zkitefly commented Feb 2, 2024

描述 | Description

现在的 下载 按钮点了还会弹出个重命名窗口,我建议直接把它砍了,另存为的时候再重命名,预览的文件夹可以设定成当前版本的模组文件夹。

image

image

原因 | Reason

减少下载模组的步骤

@Rovniced
Copy link
Contributor

Rovniced commented Feb 2, 2024

我感觉可以跳过这个选项 例如在图中这个地方,给一个选项框 直接下载至当前实例文件夹 在勾选后,点击下载就直接跳过这个页面下载,并且如果不存在多余的前置模组,就直接返回到上个搜索页面,这样更加节省时间(因为大部分人下模组基本不会存在下载到其他文件夹的情况)
image

@burningtnt
Copy link
Member

我感觉可以跳过这个选项 例如在图中这个地方,给一个选项框 直接下载至当前实例文件夹 在勾选后,点击下载就直接跳过这个页面下载,并且如果不存在多余的前置模组,就直接返回到上个搜索页面,这样更加节省时间(因为大部分人下模组基本不会存在下载到其他文件夹的情况)

点击下载即可
对于重命名的情况,我们会做处理的

@Tianzhisuo
Copy link

干脆直接给这个部分砍掉,改成友商PCL那种点击之后自动匹配当前或者对应版本Mods文件夹然后打开windows管理器进行保存到当前mods文件夹或者其他地方【与否】那个样子,现在这个总感觉有些脱裤子放屁

@wifi-left
Copy link
Contributor

如果可以弄成这种就更好了

@Tinhone
Copy link

Tinhone commented Feb 2, 2024

感觉可以把 名称 弹窗中的模组文件命名功能整合进 模组下载 弹窗,这样弹窗就会只有一个了。下载的时候留个 模组下载 弹窗这样的二次确认弹窗个人感觉也不是不行

@burningtnt
Copy link
Member

burningtnt commented Feb 10, 2024

来自 #2772 @OrzMiku

下载整合包时,依赖Mod未完全显示时,点击下载会再次弹出模组下载窗口
303582549-935b0ec1-bccf-48b3-ae92-e4500cc7c9f1

集思广益,各位可以在本 Issue 下发表对依赖显示、UI 表现的看法

@OrzMiku
Copy link

OrzMiku commented Feb 10, 2024

依赖依赖显示逻辑建议

整合包下载时,不应该显示依赖模组。

模组下载时,我有下面几种思路。

  1. 类似于PrismLauncher,会在点击下载后,弹出确认框,依赖的模组也会出现在确认框中。可以选择勾选/取消勾选依赖模组。

image

  1. 保持现有依赖列表的显示位置,但在每个列表项都添加一个复选框。当勾选复选框时,会自动下载此项依赖,已经安装的依赖可以默认勾选。
    image

  2. 采用老版本HMCL中,依赖列表的显示位置,但在每个列表项都添加一个快捷的下载按钮。
    image

建议将Release,Beta,Alpha版本的标识使用不同颜色区分

image
image

我猜测这个应该也是Release,Beta,Alpha版本的标识,如果是的话,Beta和Alpha前面依旧显示"R",应该是bug。

image

@zkitefly
Copy link
Member Author

zkitefly commented Feb 10, 2024

不好意思,你发的第一个(搜索筛选建议)和这个 issue 有什么联系吗?

建议这个单发 issue

@OrzMiku
Copy link

OrzMiku commented Feb 10, 2024

不好意思,你发的第一个和这个 issue 有什么联系吗?

建议单发 issue

不好意思,我的问题。我会将与这个issue相关的部分留下,其他内容重新发布一个issue。

@burningtnt
Copy link
Member

burningtnt commented Feb 10, 2024

类似于 Prism Launcher,会在点击下载后,弹出确认框,依赖的模组也会出现在确认框中。可以选择勾选/取消勾选依赖模组。
保持现有依赖列表的显示位置,但在每个列表项都添加一个复选框。当勾选复选框时,会自动下载此项依赖,已经安装的依赖可以默认勾选。

会有下载源数据的问题。部分 CurseForge 及 Modrinth 模组获取到的 FileID 为空,无法正确处理依赖文件,只能获取到 ProjectID,及依赖项目

采用老版本HMCL中,依赖列表的显示位置,但在每个列表项都添加一个快捷的下载按钮。

这是在开倒车。当时优化就是因为不同的模组版本的依赖不同,所以才分开显示。对于下载按钮,同上

整合包下载时,不应该显示依赖模组。
建议将Release,Beta,Alpha版本的标识使用不同颜色区分
我猜测这个应该也是Release,Beta,Alpha版本的标识,如果是的话,Beta和Alpha前面依旧显示"R",应该是bug。

感谢,已记录

@burningtnt
Copy link
Member

burningtnt commented Feb 13, 2024

今后的逻辑:

  • 移除 下载 按钮
  • 若文件名合法:添加 安装 按钮,点击后将直接下载至模组文件夹
  • 添加 重命名并安装 按钮,点击后将提示重命名窗口并下载至模组文件夹
  • 另存为 按钮将保持原先情况不变

burningtnt added a commit to burningtnt/HMCL that referenced this issue Feb 15, 2024
@burningtnt burningtnt linked a pull request Feb 15, 2024 that will close this issue
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.

7 participants