From 66bce610db7148466cc3012d0dbe93968253c053 Mon Sep 17 00:00:00 2001 From: Adam Date: Wed, 6 Mar 2019 14:53:45 -0500 Subject: [PATCH] Revert "Merge pull request #8070 from Nightfirecat/fix-ground-markers-bug" This reverts commit b6cd739e61bc9084c172294aaf3eb8ebeaf6f241, reversing changes made to 5bf4b37d1733d19ce339374b138b5e5c9eeaef26. --- .../client/plugins/groundmarkers/GroundMarkerPlugin.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/runelite-client/src/main/java/net/runelite/client/plugins/groundmarkers/GroundMarkerPlugin.java b/runelite-client/src/main/java/net/runelite/client/plugins/groundmarkers/GroundMarkerPlugin.java index 01952103c4..4aa613af7c 100644 --- a/runelite-client/src/main/java/net/runelite/client/plugins/groundmarkers/GroundMarkerPlugin.java +++ b/runelite-client/src/main/java/net/runelite/client/plugins/groundmarkers/GroundMarkerPlugin.java @@ -314,10 +314,11 @@ public class GroundMarkerPlugin extends Plugin } else { - // Remove any points on the same tile but are of a different color. - // Add a new point if no tile was removed, or if remembering tile colors is enabled, which means the marked - // tile was previously of a different color than the new tile marking. - if (!points.removeIf(p -> p.sameTile(point)) || config.rememberTileColors()) + // Remove any points on the same tile but are of a different color + points.removeIf(p -> p.sameTile(point)); + + // Add point back only if we are remembering tile colors, otherwise simply remove it + if (config.rememberTileColors()) { points.add(point); }