fix: make MoulConfigEditor properly call #saveNow

Author: lineargraph

common/src/main/java/io/github/notenoughupdates/moulconfig/gui/CloseEventListener.kt

@@ -17,10 +17,10 @@ interface CloseEventListener {
      * Called just before a voluntary close. Return [CloseAction.DENY_CLOSE] to override the close. Make sure to update
      * your state such that the user can close the gui afterward.
      */
-    fun onBeforeClose(): CloseAction
+    fun onBeforeClose(): CloseAction = CloseAction.NO_OBJECTIONS_TO_CLOSE
 
     /**
      * Called after the gui has been closed, both by the component gui, or by other actors, such as a teleport packet.
      */
-    fun onAfterClose()
-}
+    fun onAfterClose() = Unit
+}

common/src/main/java/io/github/notenoughupdates/moulconfig/gui/MoulConfigEditor.java

@@ -51,7 +51,7 @@
 import java.util.stream.Collectors;
 
 
-public class MoulConfigEditor<T extends Config> extends GuiElement {
+public class MoulConfigEditor<T extends Config> extends GuiElement implements CloseEventListener {
     private final long openedMillis;
     private final LerpingInteger optionsScroll = new LerpingInteger(0, 150);
     private final LerpingInteger categoryScroll = new LerpingInteger(0, 150);
@@ -1155,11 +1155,14 @@ private void handleKeyboardPresses() {
             if (target.getTarget() >= 0) {
                 target.setTarget(Math.max(0, target.getTarget() - 5));
             }
-        } else if (IMinecraft.INSTANCE.isKeyboardKeyDown(IMinecraft.INSTANCE.getKeyboardConstants().getEscape())) {
-            getConfigObject().saveNow();
         }
     }
 
+    @Override
+    public void onAfterClose() {
+        getConfigObject().saveNow();
+    }
+
     public boolean goToOption(@NotNull ProcessedOption option) {
         if (!setSelectedCategory(option.getCategory())) {
             search("");