Skip to content

[Enhancement] 原理图管理优化 - #6094

Open
ToobLac wants to merge 117 commits into
HMCL-dev:mainfrom
ToobLac:litematica
Open

[Enhancement] 原理图管理优化 #6094
ToobLac wants to merge 117 commits into
HMCL-dev:mainfrom
ToobLac:litematica

Conversation

@ToobLac

@ToobLac ToobLac commented May 16, 2026

Copy link
Copy Markdown
Contributor

Resolves #4052 Resolves #4518 Resolves #3957

#3597 应在本 PR 合并后并实现 NBT 修改功能后关闭

# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/versions/ModListPage.java

@Glavo Glavo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本审查建议由 GPT-5.6 Sol 生成


已将具体审查建议标注在对应代码行。

Comment thread HMCLCore/src/main/java/org/jackhuang/hmcl/schematic/SchemFile.java
Comment thread HMCLCore/src/main/java/org/jackhuang/hmcl/schematic/SchemFile.java
@github-actions github-actions Bot added the 1000+ label Jul 17, 2026
@Minecraft269

Copy link
Copy Markdown

试了一下 发现有关 #4518 的问题: 为什么不在没安装 投影 模组时直接隐藏 原理图管理 的入口? 因为测试了一下 发现哪怕是安装了投影 以及环境不支持的情况下 (神tm1.0的MC都提示要安装) 这个 安装投影 的按钮都会亮起

ToobLac added 7 commits July 25, 2026 19:59
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/nbt/NBTFileType.java
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/versions/ModListPage.java
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/versions/SchematicsPage.java
@ToobLac ToobLac mentioned this pull request Jul 27, 2026
5 tasks
ToobLac added 11 commits July 27, 2026 17:54
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/versions/ModListPage.java
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/ModListPage.java
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/SchematicsPage.java
#	HMCLCore/src/main/java/org/jackhuang/hmcl/addon/mod/ModManager.java
#	HMCLCore/src/main/java/org/jackhuang/hmcl/game/DefaultGameRepository.java
ToobLac added 5 commits August 6, 2026 12:04
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/construct/MDListCell.java
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/SchematicsPage.java
#	HMCL/src/main/resources/assets/lang/I18N_uk.properties
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/ModListPage.java
#	HMCL/src/main/java/org/jackhuang/hmcl/ui/instances/SchematicsPage.java
#	HMCLCore/src/main/java/org/jackhuang/hmcl/addon/mod/ModManager.java
#	HMCLCore/src/main/java/org/jackhuang/hmcl/game/DefaultGameRepository.java

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Static analysis found the following issues (max count 10):

import javafx.beans.property.ObjectProperty;
import javafx.beans.property.ReadOnlyObjectProperty;
import javafx.beans.property.SimpleObjectProperty;
import org.jackhuang.hmcl.schematic.LitematicaConfig;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[ERROR] [com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck]

Unused import - org.jackhuang.hmcl.schematic.LitematicaConfig.

import org.jackhuang.hmcl.schematic.LitematicaConfig;
import org.jackhuang.hmcl.task.Task;
import org.jackhuang.hmcl.util.Lang;
import org.jackhuang.hmcl.util.StringUtils;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[ERROR] [com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck]

Unused import - org.jackhuang.hmcl.util.StringUtils.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

5 participants