Multiple builtin item renderers for one item?
Juuxel opened this issue ยท 2 comments
The builtin item renderer API (#563) also allows replacing the builtin renderers of vanilla items (intended feature), but there will be a crash if multiple mods want to replace the renderer of one item.
Should we make the implementation use a Multimap
instead, and render the item with all registered builtin renderers?
I'd say hard fail if multiple mods register a renderer for the same item (but allow querying ). Maybe throw a special exception.
Rendering with all registered builtin renderers isn't a great idea since it makes things look weird.