-
Notifications
You must be signed in to change notification settings - Fork 522
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
changed flyby tgui #4600
changed flyby tgui #4600
Conversation
Co-authored-by: harryob <[email protected]>
Isn't tested. Flyby is now coded as a destination, which lasts 30 seconds instead of 200 seconds. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This was tested, it introduced a regression. Flybys we're always coded as destinations. Behind the scenes the shuttle is forced to transit to it's existing landing zone. This work was too change how the UI tells the backend how to do this. |
This reverts commit 9b31e08.
About the pull request
Slight change to the TGUI for dropship flight controls. I've had feedback that it is a bit unintuitive. This change should resolve novice POs accidentally launching flyby in a panic.
Explain why it's good for the game
Better UIs make everyone happier.
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>
tags.Changelog
🆑
ui: tweaked flyby controls to make them in line with other destinations
/:cl: