Use Overlay#getBounds for getting correct bounds

Use Overlay#getBounds for getting correct bounds in OverlayManager in case overlay overrides getBounds with custom functionality. Signed-off-by: Tomas Slusny <[email protected]>
vnord · Jul 22, 2018 · 8447326 · 8447326
1 parent d700ebc
commit 8447326
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayRenderer.java b/runelite-client/src/main/java/net/runelite/client/ui/overlay/OverlayRenderer.java
@@ -205,9 +205,9 @@ public void render(Graphics2D graphics, final OverlayLayer layer)
 				}
 
 				safeRender(client, overlay, layer, graphics, location);
-				dimension.setSize(overlay.getBounds().getSize());
+				final Rectangle bounds = overlay.getBounds();
 
-				if (dimension.width == 0 && dimension.height == 0)
+				if (bounds.isEmpty())
 				{
 					continue;
 				}
@@ -216,7 +216,7 @@ public void render(Graphics2D graphics, final OverlayLayer layer)
 				{
 					final Color previous = graphics.getColor();
 					graphics.setColor(movedOverlay == overlay ? MOVING_OVERLAY_ACTIVE_COLOR : MOVING_OVERLAY_COLOR);
-					graphics.drawRect(location.x, location.y, dimension.width - 1, dimension.height - 1);
+					graphics.draw(bounds);
 					graphics.setColor(previous);
 				}
 			}
@@ -391,7 +391,6 @@ else if (position == OverlayPosition.TOOLTIP)
 			subGraphics.setFont(FontManager.getRunescapeFont());
 		}
 
-
 		subGraphics.translate(point.x, point.y);
 		final Dimension dimension = MoreObjects.firstNonNull(overlay.render(subGraphics), new Dimension());
 		subGraphics.dispose();