FreezingMoon / AncientBeast

The Turn Based Strategy Game/eSport. Master your beasts! 🐺
https://AncientBeast.com
GNU Affero General Public License v3.0
1.65k stars 568 forks source link

regression: missing colored hexagons around inactive units #2405

Closed DreadKnight closed 1 year ago

DreadKnight commented 1 year ago

Inactive units no longer have colored hexagons around them. Regression was introduced somewhat recently. Here are older screenshots of how things should look: https://ancientbeast.com/multimedia/?id=screenshots How it looks now with the regression: Screenshot_20230708_171650

DreadKnight commented 1 year ago

Not necessarily all that bad, way more cleaner visually, but it could impact decision times now and then. HoMM3 does it like this and people play well, though I noticed that a lot of people haven't caught on that units for each player are facing a certain direction sigh

heroes_of_mm_3_profilelarge

heroes-of-might-and-magic-3-0002

DreadKnight commented 1 year ago

Issue on hold for now. Figuring out if to keep things like this or not, as I've considered this before tbh.

DreadKnight commented 1 year ago

Closing this in favor of #2406 - going for a cleaner look. Probably codebase could have used some cleaning though...