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

本地 Mod 文件夹内 Mod 图标显示异常 与 一些建议 #6

Closed
Puqns67 opened this issue Oct 4, 2021 · 5 comments
Closed

本地 Mod 文件夹内 Mod 图标显示异常 与 一些建议 #6

Puqns67 opened this issue Oct 4, 2021 · 5 comments
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers question Further information is requested

Comments

@Puqns67
Copy link

Puqns67 commented Oct 4, 2021

如图,我看他也正确识别到mod了,但是图标显示不正常
image

还有一些零碎的问题与建议:
如果两个mod下载源都有这个mod,两个下载源也都启用了,那么使用的是什么源下载呢?
建议在什么地方标记一下

关于版本回退,回退后新版本为什么要改成老版本的形态(.old),然后此时如果再次检查更新,就无法检测到更新了,如果是刻意跳过更新,那希望能在下方的提示中提示出来,例如"有 多少 个 Mod 已经的更新已被跳过"

image
本地的mod后面的那两个小图标存在与不存在两种形态区分起来有点难,特别是CurseForge的图标
CurseForge的图标可以选择把中间的火焰染红,当然不存在时为白色
但是我推荐的方案是:存在即显示,不存在则直接不显示图标

增加一个聚合搜索,若能规避掉相同的mod多次出现那更好

收藏按钮和禁用按钮希望能挪到mod名字后面

@kaniol-lck
Copy link
Owner

图标显示不正常是指?
对于mod文件本身没有包含图标的情况下,就会显示为本程序的LOGO。在我的观察中似乎确实forge mod中无图标的情况比较多,如果在其中确定存在图标,但没识别出来的情况下请再向我汇报,我对forge这边也不是很清楚。

单个mod的更新按钮旁会显示curseforge或modrinth的图标,长按也可在其他下载源中切换,更新全部的情况下,会在表格最右列显示更新来源。

版本回退成old比较方便管理,如果一部分是new一部分是old感觉管理起来会变得非常棘手。回退版本的更新检查这块可能是存在一些小问题,会修复的,目前还没有做忽略更新的功能。

确实应该改一下curseforge的图标(原图标比现在的更难认),不存在不显示也挺好的,我最初的考虑是都显示会比较整齐一些,我再试着修改一下排版。

聚合搜索:s∞n,在 #5 也提到了curseforge的搜索比较拉垮。

这两个按钮打算以悬浮显示的形式出现,也比较整齐一些。

@kaniol-lck kaniol-lck added bug Something isn't working enhancement New feature or request good first issue Good for newcomers question Further information is requested labels Oct 4, 2021
@Puqns67
Copy link
Author

Puqns67 commented Oct 4, 2021

如果图标只是从jar文件中读取的话,在jar中不存在图标文件,但是已在网络源找到此mod,为什么不直接尝试使用网络图标呢,这样不是更好

@kaniol-lck
Copy link
Owner

如果图标只是从jar文件中读取的话,在jar中不存在图标文件,但是已在网络源找到此mod,为什么不直接尝试使用网络图标呢,这样不是更好

确实,有道理

@Puqns67
Copy link
Author

Puqns67 commented Oct 8, 2021

我说的已经基本上都达到了,剩下唯一的聚合搜索也在 #5 提到了, 也合并过去了,那这个issue也就没必要了,close了,谢谢作者对Minecraft开源社区的贡献与对用户提议的支持!

@Puqns67 Puqns67 closed this as completed Oct 8, 2021
@kaniol-lck
Copy link
Owner

感谢支持!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants