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.

Sign up for GitHub

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

Jump to bottom

Moving melee targeting doesn't allow tile choice if adjacent #1084

Closed
Xymanek opened this issue Oct 25, 2021 · 0 comments · Fixed by #1106
Closed

Moving melee targeting doesn't allow tile choice if adjacent #1084

Xymanek opened this issue Oct 25, 2021 · 0 comments · Fixed by #1106
Labels
bug-basegame

Comments

@Xymanek
Copy link
Member

Xymanek commented Oct 25, 2021

Example (https://youtu.be/wqZ4M_LlnF0?t=1194):
image

Note that existing the targeting and using the pathing pawn melee attack shortcut will allow to choose the tile as normal
@robojumper robojumper changed the title Moving melee targeting doesn't allow tile choice if adjecent Moving melee targeting doesn't allow tile choice if adjacent Oct 25, 2021
Iridar added a commit to Iridar/X2WOTCCommunityHighlander that referenced this issue Dec 7, 2021
@Iridar
Draft for Issue X2CommunityCore#1084
2e8fa92
@Iridar Iridar mentioned this issue Dec 7, 2021
Merged
Iridar added a commit to Iridar/X2WOTCCommunityHighlander that referenced this issue Dec 8, 2021
@Iridar
Now able to select attack tile X2CommunityCore#1084, but can't attack…
a32d512 
… from the tile the attacker is standing on.
Iridar added a commit to Iridar/X2WOTCCommunityHighlander that referenced this issue Jan 27, 2022
@Iridar
Issue X2CommunityCore#1084 - moving melee attacks can now choose the …
d1b4137 
…attack tile for adjacent targets.
Xymanek added a commit that referenced this issue Feb 17, 2022
@Iridar @Xymanek
Issue #1084 - Allow choosing target tile for meleeing adjacent targets (
8e64963 
#1106)

* Issue #1084 - moving melee attacks can now choose the attack tile for adjacent targets.

* Implement requested changes.

* Update X2WOTCCommunityHighlander/Src/XComGame/Classes/X2MeleePathingPawn.uc

Co-authored-by: Xymanek <xymanek@outlook.com>

* Update X2WOTCCommunityHighlander/Src/XComGame/Classes/X2MeleePathingPawn.uc

Co-authored-by: Xymanek <xymanek@outlook.com>

* Fix adjacent attacks against targets on or near slopes and stairs.

* Switched to using `UnitState.GetVisibilityExtents()`

* Added UnitSize check for the attacker.

Co-authored-by: Xymanek <xymanek@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug-basegame
Projects
None yet
Milestone
No milestone
1 participant
@Xymanek