Close Request Handlers are never executed

[AzureAaron]

The closeRequestHandler Runnable inside of the GuiContext class is never executed when the screen closes rendering it ineffective for performing actions when the MoulConfig screen is closing (e.g. saving the config).

[lineargraph]

Just to be clear the field GuiContext.closeRequestHandler should allow users to request a close. If you want a listener that gets called before closes, then you should add yourself to onBeforeClose which is called on every CloseEventListener in the event hierarchy. or is the comment on the MoulConfigEditor not handling those events?

[lineargraph]

oh yeah, that MoulConfigEditor class just manually decides to close based on button presses.

[AzureAaron]

Just to be clear the field GuiContext.closeRequestHandler should allow users to request a close. If you want a listener that gets called before closes, then you should add yourself to onBeforeClose which is called on every CloseEventListener in the event hierarchy. or is the comment on the MoulConfigEditor not handling those events?

I wasn't sure what it was for, I just assumed its meant to perform an action to close the screen since MoulConfigScreenComponent set it to Screen#close though interestingly the actual runnable it created there wasn't called ever.