No callbacks for ignored slots?

[skipperguy12]

Currently you are unable to detect an ignored slot click. This can be helpful when needed to say replace the clicked item with air to allow the player to place his/her item in easily.

This issue can be addressed in 2 ways:

1. Change https://github.com/gcflames5/menu-framework/blob/master/src%2Fmain%2Fjava%2Fnet%2Fnjay%2FMenuRegistry.java#L98

to not set anything for an ignored slot by default

2. Add an IgnoredMenuItem annotation and send the callback there rather than modify MenuItem

I'm not sure which would be most developer friendly.

[AustinLMayes]

I never understood the point of ignoredSlots.. maybe just a ItemImputAnnotaion?

[gcflames5]

The point of IgnoredSlots is to provide a way to allow the user to interact with the inventory (ie. not cancelling the event in the case of that particular slot). This way, users can actually fill a slot with an item they'd like to replace, for example. Skipper is suggesting a way to detect when an item has been placed rather than through a static check after a command has been run.

[gcflames5]

As a temporary solution to this problem I would recommend a listener in the menu class that listens on that inventory so that you can detect clicks.

[AustinLMayes]

So yourse saying you can also remove iems from ignored slots? If so, maybe make that an option?

[skipperguy12]

@NathanTheBook That would be hardcoding a callback, instead it would be preferred to invoke the correct method (which would now be a callback) for when an item in an ignored slot has been clicked. This way the method can do what is needed, for example remove the item and allow a replace.