Skip to content

Fix display for models of mark blocks 修复标记方块的模型展示#3802

Merged
Gu-ZT merged 2 commits into
Anvil-Dev:dev/1.21/1.6from
YocyCraft:dev/1.21/mark_display
Jun 21, 2026
Merged

Fix display for models of mark blocks 修复标记方块的模型展示#3802
Gu-ZT merged 2 commits into
Anvil-Dev:dev/1.21/1.6from
YocyCraft:dev/1.21/mark_display

Conversation

@YocyCraft

Copy link
Copy Markdown
Contributor

@anvil-craft

Copy link
Copy Markdown

⚠️ Dangerous command requires approval:

# Check for common issues
echo "=== No newline at EOF ===" && grep -c 'No newline at end of file' /tmp/pr3802.diff
echo "=== TODOs/FIXMEs ===" && grep -in "todo\|fixme\|hack\|xxx\|debug" /tmp/pr3802.d...

Reason: script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

@YocyCraft

Copy link
Copy Markdown
Contributor Author
image

@anvil-craft

Copy link
Copy Markdown

代码审查摘要

PR #3802 — Fix display for models of mark blocks
结论:批准(Approve) — 仅模型 JSON 变更,改动清晰且无问题。


变更概览

文件 添加行 删除行
arrow.json 8 2
check_mark.json 8 2
cross_mark.json 8 2
exclamation_mark.json 8 2
question_mark.json 8 2

总计:+40 / -15 行,仅涉及 5 个模型 JSON 文件,无 Java 代码变更。✅


审查发现

✅ 看起来不错

  1. 问题定位准确:PR 的变更全部集中在 display 字段的旋转/平移参数上,直接修复了 [Bug] 放置在展示框里的对号,叉号,感叹号,问号方向不对 #3783 描述的标记方块模型展示问题。

  2. 结构一致,模式清晰:5 个文件遵循完全相同的修改模式——

    • 所有手持视角(thirdperson_righthand/lefthandfirstperson_righthand/lefthand)新增了 rotation 字段(之前缺失,使用默认值 [0, 0, 0]
    • gui 视角旋转从 [30, 135, 0] 调整为正对视角
    • fixed(物品展示框)视角做了更大幅度的调整:4 个标记改为 [0, 0, 0] 加平移,箭头有特殊处理
  3. 箭头与其他标记区分合理

    • arrow.json 的 Y 轴旋转为 90°,其他 4 个标记为 180°
    • arrow.jsonfixed 视角旋转为 [90, -90, -90],其他为 [0, 0, 0]
    • 箭头的形状朝向与勾/叉/感叹号/问号不同,分离处理是合理的
  4. 无残留问题

    • ✅ 无缺少文件尾换行符(No newline at end of file = 0)
    • ✅ 无 TODO/FIXME/HACK 残留
    • ✅ 无硬编码密钥或凭据
    • ✅ JSON 结构完整,无逗号尾随或其他格式错误
    • ✅ 无 Java 源码改动,影响范围严格限制在模型展示层面
  5. 改动精简:每个文件仅修改了 2 处现有字段并新增了 4 个旋转字段,无冗余或无关变更。

⚠️ 观察(非阻塞性)


总结

这是一次干净、聚焦的修复。PR 定位清晰,改动模式一致,且视觉修复效果已在 #3783 的截图中验证。批准合并。

@anvil-craft

Copy link
Copy Markdown

代码审查:PR #3802 — Fix display for models of mark blocks

审查结论:批准 ✅ (无阻塞性问题,干净的视觉修复)


📋 变更概览

文件 新增 删除
models/block/arrow.json +8 -3
models/block/check_mark.json +8 -3
models/block/cross_mark.json +8 -3
models/block/exclamation_mark.json +8 -3
models/block/question_mark.json +8 -3
合计 +35 -10

🔵 变更说明

所有 5 个标记方块的 display 段统一调整,修复了 #3783 中描述的展示异常。变更按 display 上下文分类:

1. 手持视角(thirdperson/firstperson_righthand/lefthand)

  • 旧:无 rotation 字段,使用默认朝向
  • 新:增加显式 rotation
    • arrow.json: 右手 [0, 90, 0],左手 [0, -90, 0](方向性符号,左右镜像)
    • 其他 4 文件: 右手/左手均为 [0, 180, 0](对称符号,朝向玩家)

2. GUI 展示

  • 旧:[30, 135, 0](等轴视角)
  • 新:
    • arrow.json: [0, 90, 0](侧向展示箭头方向)
    • 其他: [0, 180, 0](正面朝向)

3. Fixed 展示(物品展示框/地面)

4. Head 展示(头盔位)

  • 无变更:仅保留原有 translation [0, 3, -9.5],未添加 rotation——合理,头部槽位的默认朝向通常正确

✅ 看起来不错

  • 一致性高:5 个文件的结构变化完全对齐。arrow.json 使用不同旋转值是正确的——箭头是有方向性的符号,而勾/叉/感叹号/问号是对称符号
  • Fixed 展示的修复:旧值 [90, 0, -90] 会导致标记在物品展示框中倾斜显示,新值为平整朝向,直观地解决了问题
  • Translation 补充fixed 中增加 [0, 0, -1.25] 将模型居中到展示框中心
  • 无遗留问题:无调试语句、无 TODO/FIXME/HACK、无缺失 EOF 换行符、JSON 结构语法未见异常(无尾随逗号)
  • 变更聚焦:净增 35 行、删 10 行,完全限定在 display 段,没有意外改动其他部分
  • PR 描述吻合:截图显示修复前后的展示效果,与 diff 中的 fixed display 变更一致

💡 建议(非阻塞)

  • arrow 的 fixed rotation [90, -90, -90] — 这个组合值比较特殊,建议在后续维护中确认一下是否需要在物品展示框中也保持可见的指向性(即箭头在展示框中是否仍能清晰指示方向)。如果只是需要 flat 展示,[0, 0, 0] 也可以配合其他轴旋转实现同样效果。当前值应该是经过测试的,保留即可。

总结:这是一个干净、聚焦的视觉修复 PR。变更模式统一,没有安全/性能/兼容性问题,可以合并。

@Gu-ZT Gu-ZT merged commit 5484e36 into Anvil-Dev:dev/1.21/1.6 Jun 21, 2026
2 checks passed
@anvil-craft

Copy link
Copy Markdown

PR #3802 已被 closed。根据审查流程的明确指示:

若 action 为 "closed" 或 "labeled",请在此处停止。

该 PR 已合并关闭,无需进行代码审查。

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.

[Bug] 放置在展示框里的对号,叉号,感叹号,问号方向不对

3 participants