Skip to content

Fix block overlay was seen through the Interfaces block preview#429

Merged
bqc0n merged 4 commits intodevelopfrom
fix-interface-rendering
Mar 11, 2026
Merged

Fix block overlay was seen through the Interfaces block preview#429
bqc0n merged 4 commits intodevelopfrom
fix-interface-rendering

Conversation

@bqc0n
Copy link
Copy Markdown
Member

@bqc0n bqc0n commented Mar 6, 2026

ブロックの選択オーバーレイが、粘土インターフェース系ブロックのプレビューを貫通していたのを修正。

@bqc0n bqc0n requested review from Copilot and soramame0256 March 6, 2026 14:00
@bqc0n bqc0n added the bug Something isn't working label Mar 6, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a rendering issue where the vanilla block-selection overlay could appear “through” the Interface block preview by adjusting GL state handling during the highlight render.

Changes:

  • Wrap highlight rendering in GlStateTracker.pushState()/popState() to avoid leaking GL state.
  • Stop using depthMask(false) during the item-preview render so the preview writes to the depth buffer.
  • Refactor overlay box rendering to use CRenderUtils.renderCube(...) and centralize translucent/xray state setup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

bqc0n and others added 2 commits March 7, 2026 09:09
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@soramame0256 soramame0256 left a comment

Choose a reason for hiding this comment

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

lgtm

@bqc0n bqc0n merged commit 3c0d971 into develop Mar 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants