-
Notifications
You must be signed in to change notification settings - Fork 233
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
Polish area attack dragger code and visuals #6317
Conversation
@@ -51,9 +45,8 @@ end | |||
|
|||
---@param units Unit[] | |||
---@param target Vector | |||
---@param doPrint boolean # if true, prints information about the order | |||
---@param radius? number | |||
function AreaAttackOrder(units, target, doPrint, radius) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The printing was not used (yet), but in general the printing exists as a form of feedback for the user that the command is processed. I'm fine with it (not) existing, but that's why all the other special commands (distribute/copy orders, assist-to side effects, etc) have this printing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The polishing that it needed 👍 !
a54034e
to
7202fcd
Compare
Description of the proposed changes
Builds on top of/contains #6312 and #6316 (rebase this PR when those are merged in).
Testing done on the proposed changes
Use the area attack order by clicking and dragging and make sure it looks good.
Checklist