From 1ed37f561dcbf5357ab34b08c279569a73cc2a81 Mon Sep 17 00:00:00 2001 From: Max Weber Date: Sun, 15 May 2022 18:31:05 -0600 Subject: [PATCH] overlay: do not move snap points backwards because WidgetOverlays in their default position render in the vanilla position it can cause other overlays to occlude previous overlays when they should not --- .../java/net/runelite/client/ui/overlay/OverlayUtil.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayUtil.java b/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayUtil.java index e49c8c4140..a9f68d6b92 100644 --- a/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayUtil.java +++ b/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayUtil.java @@ -213,19 +213,19 @@ public class OverlayUtil switch (overlayPosition) { case BOTTOM_LEFT: - sX = bounds.x + bounds.width + padding; + sX = Math.max(sX, bounds.x + bounds.width + padding); break; case BOTTOM_RIGHT: - sX = bounds.x - padding; + sX = Math.min(sX, bounds.x - padding); break; case TOP_LEFT: case TOP_CENTER: case CANVAS_TOP_RIGHT: case TOP_RIGHT: - sY = bounds.y + bounds.height + padding; + sY = Math.max(sY, bounds.y + bounds.height + padding); break; case ABOVE_CHATBOX_RIGHT: - sY = bounds.y - padding; + sY = Math.min(sY, bounds.y - padding); break; default: throw new IllegalArgumentException();