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

Unexpected teleportation when using switch unit #330

Closed
Kexanone opened this issue Jun 29, 2018 · 2 comments · Fixed by #337
Closed

Unexpected teleportation when using switch unit #330

Kexanone opened this issue Jun 29, 2018 · 2 comments · Fixed by #337
Assignees
@Kexanone
Labels
bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time.
Milestone
1.1.1

Comments

@Kexanone
Copy link
Member

Issue (Noir Lime):

Found a bug:
When you try to switch unit, sometimes it teleports the unit to Zeus' camera position, thus killing the unit via fall damage if the camera is high.
Sometimes it happens, sometimes it doesn't. It's really inconsistent.

Possible Solution:

Well, we could change the position of the camera to the unit's position right before the switch.
@Kexanone Kexanone added bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time. labels Jun 29, 2018
@Kexanone Kexanone added this to the 1.1.1 milestone Jun 29, 2018
@Kexanone Kexanone self-assigned this Jun 29, 2018
@Kexanone Kexanone mentioned this issue Jul 13, 2018
Merged
@CreepPork
Copy link
Member

I’m still not really sure about this issue. Perhaps we should investigate the issue more before we add a “fix” that could have adverse effects.

@Kexanone
Copy link
Member Author

Kexanone commented Jul 14, 2018 •
edited
Loading

#337 has no effect for those that have no issues. So why bother? While I wasn't able to reproduce the issue, I trust Noir Lime's report. Why not add something that likely fixes what he reported, but does not change the behaviour of the module in a noticeable way?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Kexanone Kexanone

Labels
bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time.
Projects
None yet
Milestone
1.1.1
Development

Successfully merging a pull request may close this issue.

Fix unexpected teleportation of switched units ArmaAchilles/Achilles
2 participants
@CreepPork @Kexanone