feat(instance): support install optifine #1149
Conversation
|
this PR closes #959 |
xunying123
changed the title
|
测试中，我在以下多个配置下都遇到了 NO_DOWNLOAD_API 的报错： 
|
| add_library_entry(&mut client_info.libraries, &optifine_runtime_coord, None)?; | ||
| let lw_main = "net.minecraft.launchwrapper.Launch".to_string(); | ||
|
|
||
| // 是否需要考虑到已经有过了 |
There was a problem hiding this comment.
项目中请使用英文注释，如果是待讨论或待完成请使用 NOTE: 或 TODO:
| let optifine_rel = convert_library_name_to_path(&optifine_coord, None)?; | ||
| let optifine_path = libraries_dir.join(&optifine_rel); | ||
| if !installer_path.exists() { | ||
| return Err(InstanceError::LoaderNotDownloaded.into()); |
There was a problem hiding this comment.
这个错误是新引入的吗，之前 forge 和 neoforge 的 finish install 没有检查或者可复用的吗
There was a problem hiding this comment.
是新引入的，因为刚发现之前都没有检测installer是否存在，所以如果用户在下载完和执行安装之间的很短的时间把installer删掉了就会出问题
There was a problem hiding this comment.
是新引入的，因为刚发现之前都没有检测installer是否存在，所以如果用户在下载完和执行安装之间的很短的时间把installer删掉了就会出问题
那感觉可以改个名比如 LoaderInstallerNotFound 啥的？现在和 loader 的 NotDownloaded 状态有点混淆（那个应该对应重新开启下载？是吗 @Reqwey ）
btw，之前都没有检测的话，另外两个是不是也可以加一下（？
|
前端需要增设 实例-光影 页面的 加载器 section，以及更换光影加载器的 modal（复用 or 独立？） |
考虑此列固定宽度（百分比或 chakra size），考虑到文字长度可变与多语言的问题，感觉可以加 minW？保证中文下点击四个长度一样（中文最长是点击 NeoForge 时的长度，感觉挺合适的）；外语如果变长就让他们比 minW 长 |
|
另外，Optifine 类之后是不是可以改名 OptifineInfo；ModLoader 也拆出来把结构体叫 ModLoaderInfo；对应一些函数逻辑里的变量名和实际意义 Instance 结构体里的对应字段名不变（mod_loader, optifine） |
… missing toast locale
3 participants
Checklist
This PR is a ..
Description
close #959