Jake Potrebic

I think it should be yes. The goal of the global renderer is to NOT have to use resource packs. You can already use resource packs with custom translations, you...

@jpenilla > only issue with that PR is some weird stuff could happen if you use the sender after the command block is removed might want to add better exceptions/checks...

This is really confusing for the user. There are actually 11 states, 1 undamaged state, and 10 damaged states, 0-9. Using a float to set any of those 11 states...

Based on where the particles are spawned in the logic surrounding assigning job site, this is works as intended. Take the following example where a villager exists and has a...

Double reported here: https://hub.spigotmc.org/jira/browse/SPIGOT-7183

So this just essentially changes the thrown error behavior to a NOOP behavior. There is a check added by craftbukkit in side the runnable in `ensureRunningOnSameThread` that just returns out...

The `CraftItemEvent` doesn't have a "result" field, so it can't be incorrect. It has a "clicked item" field, and that is correct, you only clicked on a single item stack...

The event should probably function like shift clicking multiple stacks. Instead of trying to aggregate all the stacks crafted, just call the event once per craft. So maybe it'll be...

This is smth along the lines of what I had in mind. . Not 100% satisfied, its just a mockup pretty much. And it only handles quickmoves atm, not other...

@kezz ok, I moved the isFine, isBlock methods, and the toBlock impl. but I don't see any other methods that would benefit from that? could you clarify (if there are...