Skip to content

WIP: Port BlockUI to Minecraft 26.2 - #131

Closed
MrFruitDude wants to merge 1 commit into
ldtteam:version/26from
MrFruitDude:port/26.2-wip
Closed

WIP: Port BlockUI to Minecraft 26.2#131
MrFruitDude wants to merge 1 commit into
ldtteam:version/26from
MrFruitDude:port/26.2-wip

Conversation

@MrFruitDude

@MrFruitDude MrFruitDude commented Sep 4, 2026

Copy link
Copy Markdown

WIP / review disclaimer

This is a coordinated work-in-progress port to Minecraft 26.2. The game starts and core logic works, but crashes and texture/rendering mismatches remain under active investigation. It is not ready to merge.

Changes

  • Port BlockUI screen, control, item, fake-level, network, and rendering code to NeoForge/Minecraft 26.2.
  • Make avatar EntityIcon states render with the default AvatarRenderState instead of throwing during GUI construction.
  • Make tooltip refreshes null-safe when an item/inventory slot is cleared, avoiding an empty-slot client crash.
  • Keep block-state PiP rendering and item/material parity as active WIP follow-up work.

Change map (96 changed files)

Area Files Review focus
src/main/java 78 Screen/control migration, PiP item/block rendering, fake levels, and 26.2 GUI submission
src/main/resources 10 UI assets, atlas metadata, and mod descriptors
Gradle/version wiring 4 Java 25/NeoForge 26.2 coordinates
Tests/other 4 Codec tests and repository metadata

Key review files: controls/EntityIcon, controls/ItemIcon, mod/item/BlockStatePipRenderer, BOGuiGraphics, UiRenderMacros, and Loader.

flowchart LR
  X[XML/UI controls] --> G[BOGuiGraphics + render macros]
  G --> I[Item/entity/PiP render state]
  I --> P[NeoForge 26.2 submit pipeline]
  P --> M[MineColonies citizen + inventory screens]
Loading

Validation

  • Java 25; ./gradlew build succeeds.
  • The linked minecolonies-26.2-test profile reaches block-atlas creation with no fatal startup exception.
  • Citizen/inventory UI and all texture variants still need full in-world parity review.

TODO audit

The coordinated scan found 32 TODO/FIXME/XXX markers in tracked BlockUI source (excluding generated/build/runtime output). The avatar and empty-slot crash paths are addressed here; remaining port/render TODOs stay visible for follow-up.

Show-me / dependency graph

The review map follows the show-me skill: short prose, focused ownership boundaries, and diagrams for the dependency/control flow.

flowchart LR
  B[BlockUI 26.2] --> S[Structurize 26.2]
  S --> MC[MineColonies 26.2]
  D[Domum-Ornamentum 26.2] --> MC
  P[Piston-Unlimited 26.2] --> MC
Loading

Companion PRs

This draft is part of the coordinated MineColonies 26.2 port. Review and test these together:

Standards

Draft only; repository issue/CLA requirements and maintainer review still apply.

Base branch note

The upstream repository does not currently publish a version/26.2 branch. This PR is therefore based on the existing 26.x version/26 branch, while its head is the new port/26.2-wip branch. It should be retargeted and rebased when maintainers cut the official 26.2 base.

Do not merge this snapshot into version/26; it is only a transport/review anchor until version/26.2 exists.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

3 participants