architectury / architectury-api

An intermediary api aimed at easing development of multiplatform mods.
https://discord.architectury.dev/
GNU Lesser General Public License v3.0
306 stars 55 forks source link

[Feature] Common API for Forge IItemDecorator and Fabric DynamicItemRenderer #495

Open ShayBox opened 2 months ago

ShayBox commented 2 months ago

I believe these two APIs are very similar in what they do and could benefit from a common API (if possible).

Fuzss commented 2 months ago

Don't you mean IClientItemExtensions::getCustomRenderer as an equivalent to Fabric's DynamicItemRenderer?

ShayBox commented 2 months ago

I'm not very familiar with either API right now, if getCustomRenderer is closer to DynamicItemRender then that should be used, there's also a deprecated Fabric BuiltinItemRenderer which might help support old Fabric versions that don't have DynamicItemRenderer.