From 344453735afe5f6bacd445a877aea205feaa4787 Mon Sep 17 00:00:00 2001 From: Toad-Dev <748280+toad-dev@users.noreply.github.com> Date: Mon, 24 Jan 2022 14:19:10 -0800 Subject: [PATCH] Fix mouse_up events not firing. - deja vu --- .../computercraft/client/gui/ComputerScreenBase.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/main/java/dan200/computercraft/client/gui/ComputerScreenBase.java b/src/main/java/dan200/computercraft/client/gui/ComputerScreenBase.java index 5fdb743cd..156fffefa 100644 --- a/src/main/java/dan200/computercraft/client/gui/ComputerScreenBase.java +++ b/src/main/java/dan200/computercraft/client/gui/ComputerScreenBase.java @@ -113,6 +113,16 @@ public abstract class ComputerScreenBase extend return changed; } + // Fabric note: Forge patches some behavior in AbstractContainerScreen and so doesn't need this override. Fabric + // needs it as the vanilla mouseReleased override in AbstractContainerScreen (the super method from here) always + // consumes the event and doesn't forward it on. + @Override + public boolean mouseReleased( double x, double y, int button ) + { + return (getFocused() != null && getFocused().mouseReleased( x, y, button )) + || super.mouseReleased( x, y, button ); + } + @Override public final boolean mouseDragged( double x, double y, int button, double deltaX, double deltaY ) {