fix: improve entity rendering order determination while moving (#2760)

jcsnider · web-flow · commit 1f48c1ffa96a · 2025-11-30T18:57:36.000-03:00
diff --git a/Intersect.Client.Core/Entities/Entity.cs b/Intersect.Client.Core/Entities/Entity.cs
@@ -1179,7 +1179,14 @@ public virtual bool ShouldDrawName
                     entityY = MapHeight * 3 + Y;
                 }
 
-                entityY = (int)Math.Floor(entityY + OffsetY / TileHeight);
+                if (IsMoving && DirectionMoving == Direction.Up || DirectionMoving == Direction.UpLeft || DirectionMoving == Direction.UpRight)
+                {
+                    entityY = entityY - 1;
+                }
+                else if (IsMoving && DirectionMoving == Direction.Down || DirectionMoving == Direction.DownLeft || DirectionMoving == Direction.DownRight)
+                {
+                    entityY = entityY + 1;
+                }
 
                 if (priority >= renderingEntities.GetLength(0) || entityY >= renderingEntities.GetLength(1))
                 {

Original file line number	Diff line number	Diff line change
`@@ -1179,7 +1179,14 @@ public virtual bool ShouldDrawName`
`1179`	`1179`	`entityY = MapHeight * 3 + Y;`
`1180`	`1180`	`}`
`1181`	`1181`
`1182`		`- entityY = (int)Math.Floor(entityY + OffsetY / TileHeight);`
	`1182`	`+ if (IsMoving && DirectionMoving == Direction.Up \|\| DirectionMoving == Direction.UpLeft \|\| DirectionMoving == Direction.UpRight)`
	`1183`	`+ {`
	`1184`	`+ entityY = entityY - 1;`
	`1185`	`+ }`
	`1186`	`+ else if (IsMoving && DirectionMoving == Direction.Down \|\| DirectionMoving == Direction.DownLeft \|\| DirectionMoving == Direction.DownRight)`
	`1187`	`+ {`
	`1188`	`+ entityY = entityY + 1;`
	`1189`	`+ }`
`1183`	`1190`
`1184`	`1191`	`if (priority >= renderingEntities.GetLength(0) \|\| entityY >= renderingEntities.GetLength(1))`
`1185`	`1192`	`{`