Item Pickup Log on Armor about skyblockaddons

[DidiSkywalker]

Coins are displayed in the scoreboard next to your purse now right? I think that's pretty good and would only clutter the pickup log.
But armor slots should probably be included, yea.

[MCGamer00000]

Yes, it is included now, but it is much like the action bar system, sometimes it increases more than the s coreboard says it increases. Like Scavenger and telekinesis set the scoreboard at the same time, and only one shows.
Anyway, your right that it would clutter. Let's just forget about the suggestion.
The point of this issue is to include the armor slots.

[ViralTaco]

Also, as a (possible) suggestion, maybe make the log include coins, as the system that hypixel has for telling whenever you get coins is very bad.

That's actually due to how minecraft was made, lol.
At least for 1.8.9 from Forge. Just look at net.minecraft.item.ItemStack
While it's possible to implement it doesn't seem like the most urgent of issues.

[t-brieger]

I cant seem to manage to set up my development environment correctly so I cant test it, but that should be fixed by changing the line

main.getInventoryUtils().getInventoryDifference(p.inventory.mainInventory); //PlayerListener.java L349

to

ItemStack[] combinedInventory = new ItemStack[p.inventory.mainInventory.length + p.inventory.armorInventory.length];
System.arraycopy(p.inventory.mainInventory, 0, combinedInventory, 0, p.inventory.mainInventory.length);
System.arraycopy(p.inventory.armorInventory, 0, combinedInventory, p.inventory.mainInventory.length, p.inventory.armorInventory.length);
main.getInventoryUtils().getInventoryDifference(combinedInventory);