Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undeploy on attack ground causes the unit to move into range and stop #16673

Open
dragunoff opened this issue Jun 9, 2019 · 2 comments

Comments

Projects
None yet
4 participants
@dragunoff
Copy link
Contributor

commented Jun 9, 2019

If a deployed unit is ordered to attack ground outside of firing range it would undeploy, move into range and just stop.

fGS2AQoYDH

I'm not sure what the behavior should be here. Either one of these:

  • undeploy, move into range, deploy, attack ground
  • undeploy, move into range, attack ground (if unit is able to attack in undeployd state)
  • do not undeploy, ignore order if target is out of range (or even not display an attack cursor)

The last one seems most useful because the player probably would not want to undeploy when target is out of range (imagine a group of deployed artilery units firing on a moving infantry blob). This however goes against how the force move modifier works.

Also discussed in #16607
Related issue: #13040

@pchote

This comment has been minimized.

Copy link
Member

commented Jun 9, 2019

The third option is already implemented for standard attacks outside of range. Force attack (like when attacking ground) overrides this feature and enables out of range attacks.

The correct behaviour in this override case is the first point.

@GraionDilach

This comment has been minimized.

Copy link
Contributor

commented Jun 10, 2019

Transferring my reply from #16607 :

There is a problem with "attack ground" - if a deployed unit is ordered to attack ground outside of firing range it would undeploy, move into range and just stop.

I'm not sure what the behavior should be here. Either one of these:

  • undeploy, move into range, deploy, attack ground
  • undeploy, move into range, attack ground (if unit is able to attack in undeployd state)
  • do not undeploy, ignore order if target is out of range

The last one seems most useful because the player probably would not want to undeploy when target is out of range (imagine a group of deployed artilery units). This however goes against how the force move modifier works.

This just emphasizes the issue that Attack Ground in it's current implementation is one giant UI hack and should have been transferred to a separate button within the command bar.

@abcdefg30 abcdefg30 added the Bug label Jun 10, 2019

@pchote pchote modified the milestone: Next Release Jun 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.