Skip to content

Conversation

@PepperCode1
Copy link
Member

  • Allow SpriteGetter.spriteFinder() to return the item atlas' sprite finder
  • Fix SpriteGetter.spriteFinder() accepting definition ID instead of texture ID
  • Fix RenderLayerHelper.getEntityBlockLayer() and ItemRenderContext.getVertexConsumer() using the wrong translucent layer

Somewhat significant design changes are still necessary to accommodate the item atlas in FRAPI, but the fixes in this PR at least fix scenarios that work correctly in vanilla but not with FRAPI/Indigo.

- Allow SpriteGetter.spriteFinder() to return the item atlas' sprite finder
- Fix SpriteGetter.spriteFinder() accepting definition ID instead of texture ID
- Fix RenderLayerHelper.getEntityBlockLayer() and ItemRenderContext.getVertexConsumer() using the wrong translucent layer
@PepperCode1 PepperCode1 self-assigned this Nov 10, 2025
@PepperCode1 PepperCode1 added bug Something isn't working module: renderer indigo Pull requests and issues related to Indigo's implementation of the rendering api port Porting existing code to a different version small change priority: high High priority PRs that need review and work now. Review these first. area: rendering labels Nov 10, 2025
@modmuss50 modmuss50 merged commit c38f9e2 into FabricMC:1.21.11 Nov 11, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: rendering bug Something isn't working module: renderer indigo Pull requests and issues related to Indigo's implementation of the rendering api port Porting existing code to a different version priority: high High priority PRs that need review and work now. Review these first. small change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants