Skip to content

feat: 亮色模式、缩放设置、部分已知问题修复#141

Open
Ukiyograin wants to merge 6 commits into
FPSMasterTeam:mainfrom
Ukiyograin:main
Open

feat: 亮色模式、缩放设置、部分已知问题修复#141
Ukiyograin wants to merge 6 commits into
FPSMasterTeam:mainfrom
Ukiyograin:main

Conversation

@Ukiyograin
Copy link
Copy Markdown
Contributor

新功能和修复

首先我新增了亮色模式,并给组件缩放增加了一个设置功能,支持自定义绑定按键
7969445fd7f531627977f40487b6acf4
然后我修复了部分语言文件未更新的问题
43fe9a5d47204b3c25f0b1b852452746

以上修改都经过完整的功能测试

已知bug:

  • 组件缩放功能仍使用滚轮可直接缩放,而不需要按住默认键ctrl+滚轮,不知道怎么修复了

@Ukiyograin
Copy link
Copy Markdown
Contributor Author

以及修正了gradle仓库,避免了SSL证书问题

@Ukiyograin
Copy link
Copy Markdown
Contributor Author

image 遮罩大小重写,未启用功能颜色改为黑色,亮色模式下绑定按键和文本的框颜色一致

@Ukiyograin
Copy link
Copy Markdown
Contributor Author

还是没修好缩放设置(ctrl)

@gaoyu06
Copy link
Copy Markdown
Member

gaoyu06 commented May 31, 2026

Review 结论:暂不建议 merge,需要先处理下面几个问题。

  1. RawInput 的输入防护被弱化了
    RawInputMod / RawMouseHelper 这次删除了主线里的 acceptingInputshouldAcceptGameInput() gating,现在只在 currentScreen != null 时清 delta,不再检查 mc.inGameHasFocus。这会让窗口失焦或非游戏输入场景下的 raw mouse delta 仍可能被累计/消费,属于输入层回归风险。建议恢复主线的 focus/input gating,再在这个基础上做 active mouse 优化。

  2. 组件缩放绑定键没有真正约束组件缩放
    PR 描述里提到的已知问题仍存在:Component.display() 里仍直接读 Mouse.getDWheel() 缩放组件,没有检查 ClientSettings.zoomBind。所以打开 ClickGUI 后悬停组件时,滚轮仍会直接缩放,不需要按默认 Ctrl。建议把组件缩放也统一走新增的 zoom bind 判断,或明确拆分“组件缩放”和“UI 固定缩放”的交互设计。

  3. 不建议把 wrapper 下载和跳过 SSL 证书校验放进主线
    gradlew.bat 新增了缺失 gradle-wrapper.jar 时自动下载,并且设置 ServerCertificateValidationCallback = {$true} 跳过证书校验。这不适合进主线。建议移除这段逻辑;国内网络/镜像问题可以放到文档或本地配置里处理。gradle-wrapper.properties 切到腾讯镜像也会改变所有开发者和 CI 的依赖来源,需要项目明确接受后再合。

  4. 小问题
    git diff --check origin/main...pr/141RawMouseHelper.java EOF 多余空行,顺手清掉即可。

验证情况:GitHub checks 当前显示 build 成功;我本地用 detached worktree 跑 ./gradlew.bat build --console=plain,失败在 generateGitProperties 读取 worktree .git 路径,不是这次 Java 编译错误。建议修完以上问题后重新跑 CI,并实机验证 ClickGUI 缩放绑定、RawInput 开关和窗口失焦行为。

@gaoyu06
Copy link
Copy Markdown
Member

gaoyu06 commented May 31, 2026

另,亮色模式的部分界面我后续会做一些手动调整,暂不merge

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants