Skip to content

Conversation

@dnqbob
Copy link

@dnqbob dnqbob commented Jan 15, 2024

Life improvement
Image Update:
improve

@darkademic darkademic force-pushed the dev branch 5 times, most recently from 8063278 to 68b161a Compare January 15, 2024 22:13
@dnqbob
Copy link
Author

dnqbob commented Jan 16, 2024

The second commit: allow closest ally to be guarded when ally closest to target is too far away
examp

@darkademic darkademic force-pushed the dev branch 6 times, most recently from e12b04d to 25c5706 Compare February 10, 2024 11:29
@dnqbob dnqbob changed the title GuardsSelection Extend: Support unit will guard ally when cannot attack target under cursor GuardsSelection Extend: Unit will guard ally when cannot attack target under cursor Feb 13, 2024
@dnqbob dnqbob changed the title GuardsSelection Extend: Unit will guard ally when cannot attack target under cursor GuardsSelection Extend: Unit will guard ally when cannot attack target under cursor, without rushing to enemy's face Feb 13, 2024
@darkademic
Copy link
Owner

Hmm does this do something similar to our KeepsDistance trait? That will basically make the unit stay away from units it can't attack, when an attack order is given to other units in the same selection.

@dnqbob
Copy link
Author

dnqbob commented Feb 14, 2024

I didn't notice KeepsDistance here, feels very useful and simple as well.

@dnqbob
Copy link
Author

dnqbob commented Feb 14, 2024

Maybe a small benefit can bring in this PR compared to KeepsDistance is, when ordered to attack unattackable, healer unit will not stop healing and try move within range.

@dnqbob
Copy link
Author

dnqbob commented Feb 14, 2024

Actually, I think we can merge the function of KeepsDistance into GuardsSelection here: when there is nothing can be guarded at AttackGuard order, actor will move within range instead of doing nothing.

@dnqbob dnqbob marked this pull request as draft February 14, 2024 04:15
@dnqbob dnqbob marked this pull request as ready for review February 14, 2024 04:40
@dnqbob dnqbob force-pushed the guardselection branch 2 times, most recently from f2cbe4c to 4790871 Compare February 16, 2024 09:37
@darkademic darkademic force-pushed the dev branch 2 times, most recently from 9ab1e9f to e0ae6ff Compare February 17, 2024 20:54
@darkademic darkademic force-pushed the dev branch 3 times, most recently from c84f446 to 642ace9 Compare August 16, 2025 13:28
@darkademic darkademic force-pushed the dev branch 3 times, most recently from ee6adcb to 16f537e Compare August 30, 2025 10:43
@darkademic darkademic force-pushed the dev branch 4 times, most recently from b6fe808 to 1d583f8 Compare October 4, 2025 15:56
@darkademic darkademic force-pushed the dev branch 6 times, most recently from b9adea3 to faf82cb Compare October 18, 2025 10:20
@darkademic darkademic force-pushed the dev branch 4 times, most recently from e056164 to 68a391d Compare October 23, 2025 09:26
@darkademic darkademic force-pushed the dev branch 6 times, most recently from 4245b0b to 7a5b265 Compare November 29, 2025 12:29
@darkademic darkademic force-pushed the dev branch 3 times, most recently from e381513 to 3d06d63 Compare December 12, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants