Fixed Issue 633, Slight refactor in hexgrid to remove duplicate code

src/ui/interface.js

@@ -2428,11 +2428,14 @@ export class UI {
 			.find('.vignette.roundmarker')
 			.unbind('mouseover')
 			.unbind('mouseleave')
-			.bind('mouseover', () => {
+			.bind('mouseover', (e) => {
 				game.grid.showGrid(true);
+				game.grid.showMovementRangeInOverlay(game.activeCreature);
 			})
 			.bind('mouseleave', () => {
 				game.grid.showGrid(false);
+				game.grid.cleanOverlay();
+				game.grid.showMovementRange(game.activeCreature.id);
 			});
 
 		// Add shift keydown effect like above, onkeydown => showGrid

src/utility/hexgrid.js

@@ -1056,36 +1056,47 @@ export class HexGrid {
 
 	// TODO: Rewrite methods used here to only require the creature as an argument.
 	showMovementRange(id) {
-		let creature = this.game.creatures[id],
-			hexes;
+		let creature = this.game.creatures[id];
+		let hexes = this.findCreatureMovementHexes(creature);
 
+		// Block all hexes
+		this.forEachHex((hex) => {
+			hex.unsetReachable();
+		});
+
+		// Set reachable the given hexes
+		hexes.forEach((hex) => {
+			hex.setReachable();
+		});
+	}
+
+	showMovementRangeInOverlay(creature) {
+		let hexes = this.findCreatureMovementHexes(creature);
+
+		// Set reachable the given hexes
+		hexes.forEach((hex) => {
+			hex.overlayVisualState('hover h_player' + creature.team);
+		});
+	}
+
+	findCreatureMovementHexes(creature) {
 		if (creature.movementType() === 'flying') {
-			hexes = this.getFlyingRange(
+			return this.getFlyingRange(
 				creature.x,
 				creature.y,
 				creature.stats.movement,
 				creature.size,
 				creature.id,
 			);
 		} else {
-			hexes = this.getMovementRange(
+			return this.getMovementRange(
 				creature.x,
 				creature.y,
 				creature.stats.movement,
 				creature.size,
 				creature.id,
 			);
 		}
-
-		// Block all hexes
-		this.forEachHex((hex) => {
-			hex.unsetReachable();
-		});
-
-		// Set reachable the given hexes
-		hexes.forEach((hex) => {
-			hex.setReachable();
-		});
 	}
 
 	selectHexUp() {