Display the flight planner and the orbit analyser in the tracking station #2536
+102
−72
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These two seem to make the tracking station much more useful.
The implementation involves a bit of code restructuring where we use
PredictedVessel
systematically in many places that were usingActiveVessel
. Overall this factors out a bit of logic and fixes a buglet where the orbit analyser in the tracking station would show a stump of a window.There is a remaining know issue for a Kerbal in EVA: if a flight plan is constructed in the tracking station, that flight plan is lost when switching to the map view. That's because the Kerbal becomes unready for a couple of frames. It doesn't happen in the other direction. We call this "the invariants of KSP". The ultimate fix would be to keep some of the state of unmanaged vessels, but that's a long term effort.
Fix #2531.