Targeting active vessel also targets body it's orbiting

[Starstrider42]

Original issues at Cilph/RemoteTech2#173, Cilph/RemoteTech2#183.

May be rendered obsolete by #27.

kaufman-bryce wrote:

The feature to target the active vessel is quite nice, but could be made much more powerful if that option caused the dish to aim at the planet the active vessel is orbiting as well. Alternatively, a separate option to target active body.

HoneyFox wrote:

The idea is simple, i would like to use some pseudo-codes:
if ( activeVessel.MainBody == dishIterated.Vessel.MainBody )
fallback to "ActiveVessel" logic // P2P connection logic
else
dishIterated.setTarget( ActiveVessel.MainBody ) // Cone-angle check logic

[Starstrider42]

@RemoteTechnologiesGroup/owners, I believe that having dish cones apply consistently regardless of whether the target is a ship or celestial body (i.e., there is always a cone, and it is always centered on the current target) will make gameplay much more intuitive for new (and, apparently, not-so-new) players. I propose it be made a priority for version 1.5.0.

[pjf]

++ for @Starstrider42's comment here. That's how I thought things worked until I started targeting planets and was very surprised to see cone indicator lines appear.

[erendrake]

I agree. Very important
On Jun 28, 2014 1:43 PM, "Paul Fenwick" notifications@github.com wrote:

++ for @Starstrider42 https://github.com/Starstrider42's comment here.
That's how I thought things worked until I started targeting planets
and was very surprised to see cone indicator lines appear.

—
Reply to this email directly or view it on GitHub
#13 (comment)
.

[asdpyro]

Also agree, cones need to be cones all the time.

[ghost]

@Starstrider42 I'll see if I can get this done this weekend.

[Starstrider42]

It still needs more testing, but just to show that I've been busy:

(No, I'm not cheating -- the selected vessel has only one antenna on board.)

[Peppie84]

Merged into v1.6