Skip to content

Added reposition before the start of a game#174

Open
wellme wants to merge 5 commits into
HamaIndustries:masterfrom
wellme:formation
Open

Added reposition before the start of a game#174
wellme wants to merge 5 commits into
HamaIndustries:masterfrom
wellme:formation

Conversation

@wellme
Copy link
Copy Markdown
Collaborator

@wellme wellme commented Jul 5, 2018

Things I wanted to do but didn't:

  • Hide the enemy team in fog (Should be fairly straightforward once Fog of war bugfixing #154 gets merged, don't want to mess with it on this branch in case of conflicts), this prevents the player from reacting to the enemy's unit placement, which would reward stalling during repositioning.
  • Add an indicator that a player/team is ready and in position
  • Add a good-looking animation for units being swapped
  • Add an indicator that shows which unit is currently selected (IIRC, square under the selected unit is shown orange instead of blue
  • Use a less HACK HACK HACK way of showing which tiles are spawn points (Currently, it uses a Zone.ZoneType.MOVE_DARK, which looks okay I guess, but it could be a lot better)

The PR also introduces the OverworldInspectionContext, which is just a CursorContext that automatically manages the zones/unit information when hovering a unit. I didn't actually check if other contexts could inherit it to centralize the inspection behavior. Also: I forgot about OverworldContext.onInspectInventory. I'll fix this when I have the time.

wellme added 5 commits July 4, 2018 15:27
For some reason, applying a SwapCommand doesn't actually swap units. Oh well, here's hoping it fixes itself as I change how it fundamentally works
This allows the player to select empty spawn tiles as source/destination. Also: WHAT DID I SAY ABOUT THE ISSUE FIXING ITSELF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant